From 5818eea5543c60c07db38a0e913234b4bf7fc4db Mon Sep 17 00:00:00 2001 From: Tianon Gravi Date: Thu, 7 Aug 2025 16:42:05 -0700 Subject: [PATCH] Remove `opcache` from `php-ext-install` In 8.5, `opcache` is now built-in (and since pretty recently, it's been enabled-by-default in the PHP images too). --- test/tests/php-ext-install/container.sh | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/test/tests/php-ext-install/container.sh b/test/tests/php-ext-install/container.sh index 31b28a9b615ca..c15ad782bcc4f 100755 --- a/test/tests/php-ext-install/container.sh +++ b/test/tests/php-ext-install/container.sh @@ -5,7 +5,4 @@ docker-php-ext-install pdo_mysql 2>&1 php -d display_errors=stderr -r 'exit(extension_loaded("pdo_mysql") ? 0 : 1);' grep -q '^extension=' /usr/local/etc/php/conf.d/*pdo_mysql*.ini -# opcache is pre-built by default at least as far back as PHP 5.5 -docker-php-ext-enable opcache 2>&1 -php -d display_errors=stderr -r 'exit(extension_loaded("Zend OPcache") ? 0 : 1);' -grep -q '^zend_extension=' /usr/local/etc/php/conf.d/*opcache*.ini +# TODO now that opcache is built-in (8.5+), we could use a new zend_extension to test that they're loaded correctly too 🙈