-
Notifications
You must be signed in to change notification settings - Fork 72
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
on Windows, reading from stdin blocks the whole domain. #793
Comments
Probably needs a check for this. I see posix has one, but windows doesn't: eio/lib_eio_posix/low_level.ml Lines 50 to 53 in fdd2593
eio/lib_eio_windows/low_level.ml Lines 41 to 43 in fdd2593
|
bdodrem
added a commit
to bdodrem/eio
that referenced
this issue
Jan 25, 2025
Adding await_readable before reading fd - fix broken pipe exception : issue ocaml-multicore#792. Use Unix.read_bigarray instead of Unix_cstruct.read - replace eio_windows_cstruct_stubs.c by Unix functions. Unix.read_bigarray and Unix.write_bigarray implemented since Ocaml 5.2
bdodrem
added a commit
to bdodrem/eio
that referenced
this issue
Jan 25, 2025
Adding await_readable before reading fd * fix broken pipe exception : issue ocaml-multicore#792. Use Unix.read_bigarray instead of Unix_cstruct.read * replace eio_windows_cstruct_stubs.c by Unix functions. Since Ocaml 5.2, Unix.read_bigarray and Unix.write_bigarray can be used.
talex5
pushed a commit
to bdodrem/eio
that referenced
this issue
Jan 27, 2025
* fix blocking issue on Windows : issue ocaml-multicore#793. Adding await_readable before reading fd * fix broken pipe exception : issue ocaml-multicore#792. Use Unix.read_bigarray instead of Unix_cstruct.read * replace eio_windows_cstruct_stubs.c by Unix functions. Since OCaml 5.2, Unix.read_bigarray and Unix.write_bigarray can be used.
talex5
pushed a commit
to bdodrem/eio
that referenced
this issue
Jan 27, 2025
* fix blocking issue on Windows : issue ocaml-multicore#793. Adding await_readable before reading fd * fix broken pipe exception : issue ocaml-multicore#792. Use Unix.read_bigarray instead of Unix_cstruct.read * replace eio_windows_cstruct_stubs.c by Unix functions. Since OCaml 5.2, Unix.read_bigarray and Unix.write_bigarray can be used.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I tried to read a line from stdin with a timeout (code below). On Windows, the process blocks until I press Enter (not on Linux).
So I guess, reading the stdin on Windows blocks the whole domain.
Running the code :
blocks untill
Enter
is pressedThe text was updated successfully, but these errors were encountered: