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

new libphp7.so path #41

Closed
black-snow opened this issue Mar 9, 2018 · 9 comments
Closed

new libphp7.so path #41

black-snow opened this issue Mar 9, 2018 · 9 comments

Comments

@black-snow
Copy link

black-snow commented Mar 9, 2018

It seems like the libphp7.so will now be found at /usr/local/opt/php72/lib/httpd/modules/libphp7.so (see Homebrew/homebrew-core#25018).

Is brew-php-switcher ready for this?

@cjwilburn
Copy link
Contributor

Hi @black-snow can you confirm that you're only seeing this issue when you're updating php@7.2 with Apache? When installing without the --with-httpd flag, this appears to not be an issue.

@cjwilburn
Copy link
Contributor

It also looks like php@7.2 no longer supports the --with-httpd flag, so I'm not certain that this issue wouldn't be considered an edge case. However, I cannot speak for @philcook who is the primary maintainer of this project.

@black-snow
Copy link
Author

Most of the flags are said to be set by default for 7.2+
The fix should be rather easy, I think - we'll have to check both paths and use the first that fits.

@SMillerDev
Copy link

/usr/local/opt/php/lib/httpd/modules is perhaps a more update resilient path to use for this.

@black-snow
Copy link
Author

@SMillerDev I think that path might be used for PHP 7.3 and following in the future so we can't use it (unless we introduce a keyword like latest).

@SMillerDev
Copy link

It would be the path to the php version you selected with brew switch yeah

@black-snow
Copy link
Author

Oh really? Do we need any different paths at all then? We also don't need to touch httpd.conf then, do we? Just brew switch the version behind that and apachectl restart.

@SMillerDev
Copy link

If you only use 7.x versions and they're migrated to core I think that should be the case. No promises though 😄

ikari7789 pushed a commit to ikari7789/brew-php-switcher that referenced this issue Apr 5, 2018
@njovujsh
Copy link

njovujsh commented Dec 30, 2021

/opt/homebrew/opt/php@7.4/lib/httpd/modules/libphp7.so

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

No branches or pull requests

5 participants