-
Notifications
You must be signed in to change notification settings - Fork 26
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
Allow synchronous usage #57
Comments
I would also like to see an abstraction from Amphp (if only to have a Swoole implementation). So with that done, having a purely synchronous version should be easy. |
Can you please run @c33s I'll soon address the issues you opened in amphp. |
amphp/process#37 |
Can you confirm this update fixes the issue on Windows? |
@dantleech only partly. on windows a helper process file is needed for amphp to work, with process 1.1.1 the name of this created exe file is always the same, so it is possible to create firewall rules for it (before the filename was random, so no fw rules could be created). so it is a huge improvement. still it would be cool to use this tool without the need for creating firewall rules for helper processes. so this issue maybe should stay open. |
still requesting a synchron usage as it would help to unbreak this awesome tool on windows and also help on CI. having the colored output, no i have quite a good experience with the symfony process component (but i also use it synchron). |
the implementation of the parallel processing with on windows requires temporary file creation, where the files communicate over dynamic ports.
in a strict firewall environment (every process which want to connect has to be allowed on a specific port or port range first). currently it is not possible to handle this at all (will open a ticket at amphp).
the main problem is that the files have random names, so it is not possible to precreate firewall rules.
also see my other ticket at box-project/box#166
would love to see an
--no-parallel
flag, where your tool works synchronous without the use of amphp.The text was updated successfully, but these errors were encountered: