Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Make OPcache non-optional #18961

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Make OPcache non-optional #18961

wants to merge 4 commits into from
+161 −205

Conversation

arnaud-lb
Copy link
Member

Spun of #18660

RFC: https://wiki.php.net/rfc/make_opcache_required

Currently, OPcache is both a Zend extension and a PHP module, but it must be loaded as a Zend extension with zend_extension=opcache.so. The Zend extension then loads the PHP module. Also, OPcache can not be built statically.

This PR makes the following changes:

  • Make OPcache non-optional: OPcache is always built into the binary
  • Since the mechanism for loading builtin extensions supports PHP modules, not Zend extensions: Reversed the roles of the Zend extension and PHP module so that the PHP module loads the Zend extension.

Depends on #18939.

@arnaud-lb arnaud-lb marked this pull request as ready for review June 27, 2025 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bukka bukka Awaiting requested review from bukka bukka is a code owner

@dstogov dstogov Awaiting requested review from dstogov dstogov is a code owner

@TimWolla TimWolla Awaiting requested review from TimWolla TimWolla is a code owner

Assignees
No one assigned
Labels
Category: Build System Extension: opcache SAPI: cgi SAPI: cli SAPI: fpm SAPI: fuzzer
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@arnaud-lb @TimWolla