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
attempting to use :merge when run()ning an external command causes a core dump #5443
Comments
From @hoelzroSee the attached script. |
From @autarchuse v6; Removing the :merge eliminates the core dump. This is with the latest Rakudo on moar. |
From @MasterDuke17perl6 --version perl6 -e 'my $p = run "cat", "asdf", :err; say $p.err.slurp-rest.perl' perl6 -e 'my $p = run "cat", "asdf", :out; say $p.out.slurp-rest.perl' perl6 -e 'my $p = run "cat", "asdf", :out, :merge; say $p.out.slurp-rest.perl' perl6 -e 'my $p = run "cat", "asdf", :out, :err, :merge; say $p.out.slurp-rest.perl' perl6 -e 'my $p = run "cat", "asdf", :out, :err, :merge; say $p.err.slurp-rest.perl' perl6 -e 'my $p = run "cat", "asdf", :err, :merge; say $p.err.slurp-rest.perl' perl6 -e 'my $p = run "cat", "asdf", :merge; say $p.out.slurp-rest.perl' perl6-valgind-m -e 'my $p = run "cat", "asdf", :merge; say $p.out.slurp-rest.perl' |
From @zoffixznetFudged tests added: Raku/roast@a8afed7259 |
The RT System itself - Status changed from 'new' to 'open' |
From @zoffixznetThe test seems to hang on OSX. So any fix should look into that issue as well. The test is currently commented out in Raku/roast@4f52c6ee10 -- |
From @timoI've unsuccessfully tried to fix this, but I've come up with an idea how an actual fix might go. When the merge flag (which I added in recent commits) gets passed to MVM_proc_spawn or MVM_proc_shell, we have to first create a new pipe on our own, then set up both stdio 1 and 2 to UV_REUSE_STREAM and set the stdio_container.data.stream to the pipe we've created. |
From @ugexeResolved in rakudo/rakudo@c86090e `perl6 -e 'run("ls", :merge).out.slurp.say'` |
From @dogbert17On Wed, 12 Jul 2017 15:30:33 -0700, ugexe@cpan.org wrote:
Worls on OSX as well. The fudged test was unfudged by ugexe++. Closing issue. |
@dogbert17 - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#128594 (status was 'resolved')
Searchable as RT128594$
The text was updated successfully, but these errors were encountered: