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

Support Microsoft Windows #18

Closed
clue opened this issue May 17, 2015 · 3 comments · Fixed by #54
Closed

Support Microsoft Windows #18

clue opened this issue May 17, 2015 · 3 comments · Fixed by #54

Comments

@clue
Copy link
Owner

clue commented May 17, 2015

No description provided.

@clue
Copy link
Owner Author

clue commented Jun 12, 2016

See also #26, as MS Windows doesn't use UTF-8 by default.

@clue
Copy link
Owner Author

clue commented Jun 12, 2016

There does not appear to be a sane way to read individual key presses on MS Windows, see also #16.

In particular, the loop will be blocked once the user hits any key on the keyboard.

https://bugs.php.net/bug.php?id=34972

@clue clue closed this as completed in #54 Oct 31, 2017
@clue clue changed the title Support MS Windows Support Microsoft Windows Oct 31, 2017
@clue
Copy link
Owner Author

clue commented Oct 31, 2017

Note that Microsoft Windows is not supported.
Due to platform inconsistencies, PHP does not provide support for reading from
standard console input without blocking.
Unfortunately, until the underlying PHP feature request is implemented (which
is unlikely to happen any time soon), there's little we can do in this library.
A work-around for this remains unknown.
Your only option would be to entirely
disable interactive input for Microsoft Windows.
However this package does work on Windows Subsystem for Linux
(or WSL) without issues. We suggest installing WSL
when you want to run this package on Windows.

There's hope that https://bugs.php.net/bug.php?id=34972 will be implemented eventually. Until this happens, there's little we can do here.

This library will (have to) throw an Exception on Microsoft Windows in a future version. This is required in order to update our dependencies (#50) and eventually include reactphp/stream#46.

Relevant documentation to the README has been added via #54, so this ticket has been closed.

If anybody happens to find a work-around, please report back to this ticket and I'm happy to reopen 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant