Regression #14488
closed
Extensions directory is not set in ``rc.php_ini_setup``
100%
Description
rc.php_ini_setup not checking php version 8.2
--- /root/rc.php_ini_setup 2023-06-19 04:10:57.592644000 0000
++ /etc/rc.php_ini_setup 2023-06-19 04:10:32.037813000 +0000@ -24,7 +24,9 @
VERSION=$(/bin/cat /etc/version)
MIN_REALMEM_FOR_OPCACHE=512
-if /usr/local/sbin/pkg-static info -e php81; then
if /usr/local/sbin/pkg-static info -e php82; then
EXTENSIONSDIR="/usr/local/lib/php/20220829/"
+elif /usr/local/sbin/pkg-static info -e php81; then
EXTENSIONSDIR="/usr/local/lib/php/20210902/"
elif /usr/local/sbin/pkg-static info -e php74; then
EXTENSIONSDIR="/usr/local/lib/php/20190902/"
Updated by Marcello Silva Coutinho 11 months ago
Updated by Jim Pingle 11 months ago
I'm not sure we even need to set this anymore. It is using the correct directory already by default.
: php-config --extension-dir /usr/local/lib/php/20220829
Updated by Marcos M 6 months ago
- Tracker changed from Bug to Regression
- Subject changed from rc.php_ini_setup not checking php version 8.2 to rc.php_ini_setup extensions directory is not set
- Status changed from New to Pull Request Review
- Assignee set to Marcos M
- Target version set to 2.8.0
- Plus Target Version set to 24.03
Looks like extension_dir defaults to the correct path when the value is empty. However, EXTENSIONSDIR is used when loading modules which are not "compiled into php statically"; I'm not sure if there's a case where that would happen normally, but the fix is simple enough to include.
Updated by Jim Pingle 3 months ago
- Subject changed from rc.php_ini_setup extensions directory is not set to Extensions directory is not set in ``rc.php_ini_setup``
Updated by Jim Pingle 2 months ago
- Status changed from Feedback to New
The change here hardcoded checks for php81 and php82, but we are now on php83. Does this need another adjustment?
I'm not seeing EXTENSIONSDIR in php.ini on a current snapshot.
Updated by Jim Pingle 2 months ago
- Assignee changed from Marcos M to Jim Pingle
Yes, it does need a change for PHP 8.3. Commit pushed and should show up momentarily.
Updated by Jim Pingle 2 months ago
- Status changed from New to Feedback
- % Done changed from 0 to 100
Applied in changeset 903c1898973f3524e50ac44306838ccc6ed3558e.
Updated by Jim Pingle about 2 months ago
- Status changed from Feedback to Resolved
It's present and working on the current snapshot.