-
-
Notifications
You must be signed in to change notification settings - Fork 473
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
Fixed creating a new instance of a Process in Windows #820
base: main
Are you sure you want to change the base?
Conversation
This change will resolve a problem that occurs when creating a new process in Windows. Environment variables (PATH) are not passed properly to the Process class instance.
If these new parammeters in Process are not set, this error will occurs on Windows: In short, it is due to the fact that the environment variables must be passed correctly to create a new instance of the Symfony Process class. Setting NODE_PATH="C:\path\to\node.exe", PATH="${PATH}", setIncludePath, setNodeBinary, etc is not enough. |
Log:
|
Seems like this makes the tests fail. Could you take a look? |
Hello, Freek! Sure! |
This change will resolve a problem that occurs when creating a new process in Windows. Environment variables (PATH) are not passed properly to the Process class instance.