-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
$PATH
is changed during patch
execution
#38
Comments
Interesting. I took a quick look at this in the same way on a RH-flavor Linux install and my PATHs are not rearranged like you're describing.
Unfortunately I don't have a Mac to look into this further on that end. |
Hmm it's probably the Login directive: #!/bin/sh -l Maybe it doesn't play well with Composer's script wrapping? |
I noticed that change and wondered myself too. I did look closely at the two given PATHs and they are simply just rearranged, not actually different in content, which makes me think it's a sourcing order issue |
I guess macOS does something. |
More to your original point, it would seem advantageous in multi-version environments to be able to pass the desired PHP executable path into patch via CLI option |
This may be an issue on macOS,
$PATH
is changed insrc/patch
.So it may change the
php
binary version, and may install unexpected versions of the packages.In my case, I have both
/usr/local/opt/php@8.2/bin/php
(8.2) and/usr/local/bin/php
(8.3).I expect that PHP 8.2 is used, but patch uses PHP 8.3.
The text was updated successfully, but these errors were encountered: