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
$*IN.getc not blocking on macOS #6631
Comments
From @bbkrRakudo version 2017.09 built on MoarVM version 2017.09.1 Following code does not wait for user input but exits immediately. perl6 -e 'say $*IN.getc' BTW: This affects modules such as Terminal::Print that are unable to react to user input on macOS. My default TTY settings are: But even setting it to raw mode: does not wait for user input. |
From @bbkrI've tested previous Rakudo builds and moar-2017.05 works fine, moar-2017.07 is broken. So regression happened somewhere between those two versions. |
From @timocan you get us strace output for this? |
The RT System itself - Status changed from 'new' to 'open' |
From @bbkrThere is no strace on macOS, I used dtruss (dtrace): $ dtruss -p 1827 |
From @AlexDaniel[Coke]++ bisected it to rakudo/rakudo@80bbfcd On 2017-10-22 12:25:07, bbkr@post.pl wrote:
|
From @cokeOn Sun, 22 Oct 2017 12:25:07 -0700, bbkr@post.pl wrote:
This behavior changed on rakudo/rakudo@80bbfcd commit 80bbfcdd47bcb27c21352a53a5156a6ecdd41e65 Use same slow-path for getc and readchars. This will allow fixing the common problem they have near the end of IRC chat noted the behavior of perl6 -e '$*IN.eof.say' Is different on OS X and other platforms (True on mac, False elsewhere) -- |
From @zoffixznetOn Mon, 23 Oct 2017 07:15:56 -0700, coke wrote:
More IRC discussion: https://irclog.perlgeek.de/perl6-dev/2017-10-23#i_15341428 Kinda boils down to this snippet saying "STDIN IS: seekable" on Macos, while NOT seekable everywhere else: #include <stdio.h> And some C++ stackoverflow post claimed this was a bug in standard library on macos that was fixed in some version already. |
From @bbkr
I'll try to get you Sierra (10.12) and High Sierra (10.13) results tomorrow. |
From @zoffixznetOn Mon, 23 Oct 2017 09:28:47 -0700, bbkr@post.pl wrote:
Added a test (in a branch) that covers the bug: Raku/roast@27833272b4 However, I've no idea if that fix is sane and whether it has significant performance impact |
From @zoffixznetOn Mon, 23 Oct 2017 20:05:33 -0700, cpan@zoffix.com wrote:
Thank you for the report. This is now fixed. Fix: rakudo/rakudo@eb1febd5658377a |
@zoffixznet - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#132349 (status was 'resolved')
Searchable as RT132349$
The text was updated successfully, but these errors were encountered: