homebrew-core icon indicating copy to clipboard operation
homebrew-core copied to clipboard

php: add intl extension back to the php formula

Open shivammathur opened this issue 3 months ago • 3 comments

  • [X] Have you followed the guidelines for contributing?
  • [X] Have you ensured that your commits follow the commit style guide?
  • [X] Have you checked that there aren't other open pull requests for the same formula update/change?
  • [X] Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • [X] Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • [X] Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

  • As we have switched back to clang for PHP 8.5, it makes sense to keep intl extension part of the php formula. It made sense as a separate formula as it needed clang and building php with gcc had considerable performance gains, but now that we are using clang again with tailcall VM, there is no real benefit to keep it in separate formula, it causes various issues.
  • Moving a single extension to a separate formula creates confusion, breaking changes and adds maintenance burden. I like the idea in https://github.com/Homebrew/homebrew-core/issues/206404, but then all PHP extensions that can be build as shared should be moved to separate formulae, instead of moving just one, and such change would be part of brew's documentation and not just a caveat.
  • For the 8.5.0 release, as per the package stats around 96% of PHP installs from the core tap do not have intl, and I think many of them might have missed the caveat and not installed the php-intl package.

shivammathur avatar Nov 26 '25 03:11 shivammathur

@cho-m What are your thoughts on this?

shivammathur avatar Dec 02 '25 14:12 shivammathur

so does this mean php8.4 is the only minor version which would require the separate php-intl package?

If so, then the php-intl formular should be adjusted to only depend on [email protected]

LordSimal avatar Dec 02 '25 21:12 LordSimal

@LordSimal [email protected] no longer needs a seperate formula for intl after https://github.com/Homebrew/homebrew-core/pull/255291.

shivammathur avatar Dec 03 '25 02:12 shivammathur