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

Change PHP executable find order (and handle PHP_PATH options) #11

Closed
wants to merge 1 commit into from
Closed

Conversation

Maxeee09
Copy link

Change PHP executable detection order to allow PHP_PATH environment variable usage and override PHP_BINARY detection behavior. Specifically when you need to use a different php.ini.

@tompizmor
Copy link

Hi,

I noticed that this commit will solve an issue on OS X 10.11 El Capitan. The exact issue is that El Capitan added a new “System Integrity Protection (SIP)” that reduces the attack surface for malware that relies on modifying system files.
In some cases could be necessary to run PHP wrappers and with the current detection order is impossible to specify the PHP binary to execute.

[1]
https://www.apple.com/osx/all-features/pdf/osx_elcapitan_core_technologies_overview.pdf
See page 16 “System Integrity Protection"

Tomas

@nicolas-grekas
Copy link
Member

Thanks for this PR. Yet I invite you to submit it on the main symfony/symfony repository: this is where merges happen. Note that you should add test cases while doing so. Cheers.

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