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

Piping from one Proc to another occasionally dies with "his process is not opened for write" #6349

Closed
p6rt opened this issue Jun 18, 2017 · 5 comments
Labels
ASYNC regression Issue did not exist previously

Comments

@p6rt
Copy link

p6rt commented Jun 18, 2017

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

Searchable as RT131592$

@p6rt
Copy link
Author

p6rt commented Jun 18, 2017

From @zoffixznet

  zoffix@​VirtualBox~/CPANPRC/rakudo (nom)$ cat bar.p6
  for ^4000 {
  my $p = run(​:out, :bin, 'ls'); run(​:in($p.out), 'true')
  }
  zoffix@​VirtualBox~/CPANPRC/rakudo (nom)$ ./perl6 bar.p6
  Unhandled exception in code scheduled on thread 12
  Tried to get the result of a broken Promise

  Original exception​:
  This process is not opened for write

  zoffix@​VirtualBox~/CPANPRC/rakudo (nom)$

@p6rt
Copy link
Author

p6rt commented Jun 18, 2017

From @zoffixznet

P.S.​: the test for this ticket already exists as test #​2 in t/spec/S17-procasync/stress.t

It's an old test that now flops once in a while.

@p6rt
Copy link
Author

p6rt commented Jun 27, 2017

From @jnthn

On Sat, 17 Jun 2017 22​:18​:31 -0700, cpan@​zoffix.com wrote​:

zoffix@​VirtualBox~/CPANPRC/rakudo \(nom\)$ cat bar\.p6
for ^4000 \{
    my $p = run\(​:out, :bin, 'ls'\); run\(​:in\($p\.out\), 'true'\)
\}
zoffix@​VirtualBox~/CPANPRC/rakudo \(nom\)$ \./perl6 bar\.p6 
Unhandled exception in code scheduled on thread 12
Tried to get the result of a broken Promise

Original exception​:
    This process is not opened for write

zoffix@​VirtualBox~/CPANPRC/rakudo \(nom\)$ 

Fixed by recent changes to handle code like this by plumbing descriptors together. A further fix has made it not end up running out of descriptors also.

@p6rt
Copy link
Author

p6rt commented Jun 27, 2017

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

@p6rt p6rt closed this as completed Jun 27, 2017
@p6rt
Copy link
Author

p6rt commented Jun 27, 2017

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

@p6rt p6rt added ASYNC regression Issue did not exist previously labels Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ASYNC regression Issue did not exist previously
Projects
None yet
Development

No branches or pull requests

1 participant