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

Proc::Async sends empty trying to stderr/stdout tap on program exit, even though the program did not send anything #5871

Closed
p6rt opened this issue Dec 7, 2016 · 5 comments

Comments

@p6rt
Copy link

p6rt commented Dec 7, 2016

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

Searchable as RT130291$

@p6rt
Copy link
Author

p6rt commented Dec 7, 2016

From alexander.kiryuhin@gmail.com

The original example to reproduce​:

use v6;

my $p = Proc​::Async.new('ls', '-l');

$p.stdout.tap(-> $v { say $v });
$p.stderr.tap(-> $v { say $v.WHAT; say $v.perl; }); # Prints (Str) and
empty string..

await $p.start;

Stdout and stderr is tapped with empty output on any command.

@p6rt
Copy link
Author

p6rt commented Dec 8, 2016

From @zoffixznet

On Wed, 07 Dec 2016 07​:24​:04 -0800, alexander.kiryuhin@​gmail.com wrote​:

The original example to reproduce​:

use v6;

my $p = Proc​::Async.new('ls', '-l');

$p.stdout.tap(-> $v { say $v });
$p.stderr.tap(-> $v { say $v.WHAT; say $v.perl; }); # Prints (Str) and
empty string..

await $p.start;

Stdout and stderr is tapped with empty output on any command.

Fixed in rakudo/rakudo@08ccbbb

Marking Tests Needed

@p6rt
Copy link
Author

p6rt commented Dec 8, 2016

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

@p6rt
Copy link
Author

p6rt commented Dec 8, 2016

From @zoffixznet

On Thu, 08 Dec 2016 05​:50​:37 -0800, cpan@​zoffix.com wrote​:

On Wed, 07 Dec 2016 07​:24​:04 -0800, alexander.kiryuhin@​gmail.com
wrote​:

The original example to reproduce​:

use v6;

my $p = Proc​::Async.new('ls', '-l');

$p.stdout.tap(-> $v { say $v });
$p.stderr.tap(-> $v { say $v.WHAT; say $v.perl; }); # Prints (Str)
and
empty string..

await $p.start;

Stdout and stderr is tapped with empty output on any command.

Fixed in
rakudo/rakudo@08ccbbb

Marking Tests Needed

There are now tests for this in Raku/roast@c96e321

Closing.

@p6rt
Copy link
Author

p6rt commented Dec 8, 2016

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

@p6rt p6rt closed this as completed Dec 8, 2016
@p6rt p6rt added the testneeded label Jan 5, 2020
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