Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Setting non-strings into %*ENV breaks Proc::Async #4498

Closed
p6rt opened this issue Sep 2, 2015 · 7 comments
Closed

Setting non-strings into %*ENV breaks Proc::Async #4498

p6rt opened this issue Sep 2, 2015 · 7 comments

Comments

@p6rt
Copy link

p6rt commented Sep 2, 2015

Migrated from rt.perl.org#125953 (status was 'resolved')

Searchable as RT125953$

@p6rt
Copy link
Author

p6rt commented Sep 2, 2015

From @zostay

Here's a small broken program​:

perl6 -e '%*ENV<BROKE> = 1; shell("echo")'

The error is​:

This type cannot unbox to a native string
  in block <unit> at -e​:1
--
Sterling Hanenkamp
http://sterling.hanenkamp.com/stfl/
785-370-4454

@p6rt
Copy link
Author

p6rt commented Oct 25, 2015

From @AlexDaniel

All these are affected​:
%*ENV<FOOBAR> = 1; qx/echo/;
%*ENV<FOOBAR> = 1; run('echo');
%*ENV<FOOBAR> = 1; shell('echo');
%*ENV<FOOBAR> = 1; my $p = Proc​::Async.new('echo'); $p.start;

Same error.

Perhaps https://rt.perl.org/Public/Bug/Display.html?id=126447 has the same
roots.

@p6rt
Copy link
Author

p6rt commented Oct 25, 2015

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Oct 25, 2015

From @lizmat

Fixed with b0b2489 , tests needed

On 02 Sep 2015, at 03​:12, hanenkamp (via RT) <perl6-bugs-followup@​perl.org> wrote​:

# New Ticket Created by hanenkamp
# Please include the string​: [perl #​125953]
# in the subject line of all future correspondence about this issue.
# <URL​: https://rt-archive.perl.org/perl6/Ticket/Display.html?id=125953 >

Here's a small broken program​:

perl6 -e '%*ENV<BROKE> = 1; shell("echo")'

The error is​:

This type cannot unbox to a native string
in block <unit> at -e​:1
--
Sterling Hanenkamp
http://sterling.hanenkamp.com/stfl/
785-370-4454

@p6rt
Copy link
Author

p6rt commented Oct 25, 2015

From @usev6

I added a simple test to S02-magicals/env.t with commit Raku/roast@e58cb30a90.

I'm closing this ticket as 'resolved'. If you think that we need more tests, feel free to add to roast or reopen the ticket.

1 similar comment
@p6rt
Copy link
Author

p6rt commented Oct 25, 2015

From @usev6

I added a simple test to S02-magicals/env.t with commit Raku/roast@e58cb30a90.

I'm closing this ticket as 'resolved'. If you think that we need more tests, feel free to add to roast or reopen the ticket.

@p6rt p6rt closed this as completed Oct 25, 2015
@p6rt
Copy link
Author

p6rt commented Oct 25, 2015

@usev6 - Status changed from 'open' to 'resolved'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant