The following patch changes the "configured" wording to "enabled" for
OPcache in the "Extension modules" section of the pkg-readme. Using
"configured" made it confusing to me because there doesn't seem to be
anything that needs to be done, if the defaults are okay, other than
enabling the extension as mentioned later in the section.

For what it's worth, I tested disabling and enabling OPcache without
tweaking anything else, and phpinfo() looked fine. I was also getting
around 2x difference between the two configurations when loading the
same page. I looked at the php.ini-* patches in CVS and checked the PHP
changelogs/migration guides all the way back to 7.0, and there doesn't
seem to be a reason to require anything other than the default
configuration.

On an unrelated but PHP-related note, I recently did a pkg_add -u on a
6.6 server and got the following misleading output after PHP was
upgraded from 7.3.15 to 7.3.16:

# pkg_add -u
[snip]
--- -php-7.3.15 -------------------
You should also run rm -f /etc/php-7.3.sample/*
You should also run rm -f /etc/php-fpm.d/*
File /etc/php-7.3.sample/opcache.ini does not exist
You should also check /etc/php-7.3.ini (which was modified)

Running the first rm -f would remove the "sample" extension
configuration files, break the symlinks to them that are in /etc/php-73,
and cause those extensions to not be loaded. The other rm -f would
remove additional configuration files for php-fpm. I have a feeling
they're displayed because that package was deleted before/after the new
one version was installed, but they probably shouldn't be displayed
during an upgrade. I'm not sure if there's something that can be done in
the port or if it's just a bug/quirk in how pkg_add works.

Tim

Index: lang/php/files/README-main
===================================================================
RCS file: /cvs/ports/lang/php/files/README-main,v
retrieving revision 1.9
diff -u -p -r1.9 README-main
--- lang/php/files/README-main    29 Jan 2019 20:25:15 -0000    1.9
+++ lang/php/files/README-main    9 Apr 2020 22:37:11 -0000
@@ -82,7 +82,7 @@ in several classes.
 - some extensions are included in the main PHP package and are always
 enabled; they don't need to be installed or enabled separately.

-- opcache is in the main package but must be configured.
+- opcache is in the main package but must be enabled.

 - some 'core' extensions with extra dependencies are packaged separately
 (e.g. php-pdo_mysql, php-ldap, php-soap, and others) and can be installed

Reply via email to