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

Cannot read from STDIN in a thread on windows console #3719

Closed
p6rt opened this issue Mar 6, 2015 · 8 comments
Closed

Cannot read from STDIN in a thread on windows console #3719

p6rt opened this issue Mar 6, 2015 · 8 comments
Labels

Comments

@p6rt
Copy link

p6rt commented Mar 6, 2015

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

Searchable as RT124005$

@p6rt
Copy link
Author

p6rt commented Mar 6, 2015

From andrew@egeler.us

<retupmoca> jnthn​: is it known that "await start { .say for lines }" breaks on windows console (with perl6-m)?
<retupmoca> jnthn​: or should I rakudobug?
<J-L> On my Windows, perl6-m -e "await start { .say for lines }" gives me​: ===SORRY!=== and I have to CTRL-C to get out.
<jnthn> retupmoca​: Wasn't known to me, but I can reproduce it

Expected result​: Same behavior as “.say for lines” (without threading)

@p6rt
Copy link
Author

p6rt commented Mar 8, 2015

From @smls

On Linux, I don't get that "expected result" either.

It does not print an error, but simply does nothing, i.e.
- doesn't echo STDIN lines
- doesn't quit when STDIN is closed with [ctrl]+d

Simple ".say for lines" without the "await start { }" does both of those things.

@p6rt
Copy link
Author

p6rt commented Mar 8, 2015

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

@p6rt
Copy link
Author

p6rt commented Nov 20, 2015

From @nwc10

MoarVM work towards fixing this bug is on the MoarVM branch sync-without-uv

@p6rt
Copy link
Author

p6rt commented Dec 16, 2015

From @jnthn

On Fri Nov 20 10​:21​:26 2015, nicholas wrote​:

MoarVM work towards fixing this bug is on the MoarVM branch sync-without-uv

Having already ran into various "fun" platform inconsistencies working on that branch, I've concluded that - while I agree this issue wants fixing - it's not something that it's sensible to try and sneak in this side of the Christmas release, since it risks destabilizing a bunch of things that work fine, for the sake of a more rare thing that does not. So, it's off the xmas tickets list. It's pretty much certain that nobody will write code that relies on this failing to work, so fixing it after xmas and making it something you can expect to work in 6.d (and so including it in the test suite then) will be fine.

Note that while this ticket doesn't mention it, the issue also exists for synchronous sockets; there, though, there's an easy workaround, which is to use IO​::Socket​::Async, so it's even less of an issue.

@p6rt
Copy link
Author

p6rt commented Aug 19, 2016

From @zoffixznet

Tests appear to exist in t/spec/S16-io/handles-between-threads.t

@p6rt
Copy link
Author

p6rt commented Sep 12, 2017

From @jnthn

On Fri, 19 Aug 2016 12​:04​:15 -0700, cpan@​zoffix.com wrote​:

Tests appear to exist in t/spec/S16-io/handles-between-threads.t

And have been fully passing since earlier this year when the issue of handles between threads was addressed; resolving.

@p6rt
Copy link
Author

p6rt commented Sep 12, 2017

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

@p6rt p6rt closed this as completed Sep 12, 2017
@p6rt p6rt added the conc label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant