-
-
Notifications
You must be signed in to change notification settings - Fork 12.4k
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
php@7.4 no longer works after icu4c 72 upgrade #120854
Comments
Apologies, but I'm not familiar with how to request "please rebuild and push formula X now that a dependency has changed", and I couldn't find it in discussions or other issues from a bit of searching. Hopefully this is sufficient? If not, I'm happy to change the request. |
Not a maintainer, but "disabled on Homebrew core" means no one can install it any longer from Homebrew core, so I'm pretty sure the maintainers aren't entertaining any rebottling requests for this. Your fork means you're responsible for maintaining it going forward. |
Broadly I agree... but it is still in core, and it still receives updates, many times long after it was versioned and disabled. If it's unsupported it should probably be removed, in favor of a tap or something. Also I'm honestly not sure how to add versioning to icu4c to properly manage this (I would prefer to link it against a keg-only old ICU and just freeze everything until we can drop php7), as it fails on the bottle manifest request. From some diving around docs, I get the impression that this is a rather large knowledge cliff and I'm not even sure where to start climbing - nearly all docs assume it's a core formula. |
It's disabled because upstream no longer releases updates: https://www.php.net/supported-versions.php. I believe current policy is to permanently delete disabled formulae one year after, to give people time to copy the formula over to their own taps. At least one person is actively doing so: https://github.com/shivammathur/homebrew-php
The simplest way forward, assuming you don't want to rely on Shivam's own efforts, is to remove the |
PHP 7.4 is unsupported so we don't rebottle it for dependencies anymore |
Yep, rebuilding from source does work. It's a fine though slow workaround for now. Thanks all! I'll figure something out. |
@Groxx: Perhaps I had a similar problem. Have a mac mini late 2014 with Monterey and homebrew with php@7.4 (webserver). Yesterday I did "brew upgrade" and did not expect any problem. Afterwords php@7.4 did no longer work. Similar warning as you described. What I finally did is:
I know I have a slightly damaged brew-installation now. I will investigate what I will do to fix it. The mac mini is only a fall-back-device, so there is some time hopefully. Kind regards from Aachen, Germany |
The better fix would be to install from https://github.com/shivammathur/homebrew-php who still supports versions of php that upstream doesn't. |
If I was meant, I did not touch my installed php at all, I think. Only the webserver apache couldn't start due to php@7.4 did not find icu4c@71.1, because it was overwritten by the brew upgrade command. I think as soon as I delete the created symlink, renamed the original one to its original name the homebrew-installation is valid again. |
Thanks @SMillerDev and others above for pointing out the right way to solve this problem. |
I just applied the following on an Apple M1 Macbook Pro with Ventura 13.2:
Now it seems as if php@7.4 and php@8.1 both can work with icu4c 72.1.
|
Thank you @virk It worked like a charm. |
brew gist-logs <formula>
link ORbrew config
ANDbrew doctor
outputVerification
brew doctor
output" saysYour system is ready to brew.
and am still able to reproduce my issue.brew update
and am still able to reproduce my issue.brew doctor
and that did not fix my problem.What were you trying to do (and why)?
Install and use
php@7.4.rb
with only these changes so it can be installed and linked (we maintain a fork for this formula):What happened (include all command output)?
The latest PHP 8 works as it links against icu4c v72:
but PHP 7.4 still links to icu4c 71:
which results in the install failing:
brew install php@7.4.rb -s
works with 72 onarc
output at least, seems like it probably just needs a new build to be pushed.What did you expect to happen?
working install of 7.4
Step-by-step reproduction instructions (by running
brew
commands)Apply this patch to the latest php@7.4(.33):
and install it:
The text was updated successfully, but these errors were encountered: