ZTS support on Alpine is broken

[mvorisek]

Description

see docker-library/php#1076, the issue is described at docker-library/php#1076 (comment)

I expect all bundled exts to be ZTS compatible on every linux platform, incl. Alpine.

PHP Version

8.0, 8.1, probably also master

[cmb69]

I think we need to enable the static TSRM cache:

 ext/pdo/config.m4 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ext/pdo/config.m4 b/ext/pdo/config.m4
index 9b9a3e36df..63a2f7ed0f 100644
--- a/ext/pdo/config.m4
+++ b/ext/pdo/config.m4
@@ -9,7 +9,7 @@ if test "$PHP_PDO" != "no"; then
   dnl Make sure $PHP_PDO is 'yes' when it's not 'no' :)
   PHP_PDO=yes
 
-  PHP_NEW_EXTENSION(pdo, pdo.c pdo_dbh.c pdo_stmt.c pdo_sql_parser.c pdo_sqlstate.c, $ext_shared)
+  PHP_NEW_EXTENSION(pdo, pdo.c pdo_dbh.c pdo_stmt.c pdo_sql_parser.c pdo_sqlstate.c, $ext_shared,,-DZEND_ENABLE_STATIC_TSRMLS_CACHE=1)
   PHP_ADD_EXTENSION_DEP(pdo, spl, true)
   PHP_INSTALL_HEADERS(ext/pdo, [php_pdo.h php_pdo_driver.h php_pdo_error.h])

Interestingly:

php-src/ext/pdo/config.w32

Line 6 in 83d96d6

EXTENSION('pdo', 'pdo.c pdo_dbh.c pdo_stmt.c pdo_sql_parser.c pdo_sqlstate.c', false /* force static, PHP_PDO_SHARED is broken yet somehow */);

[unix-world]

Not sure, but as far as I know Alpine linux does not use the standard glibc ... Perhaps that's the problem there.
https://wiki.alpinelinux.org/wiki/Running_glibc_programs

Because of these there are many programs that have not yet been ported natively to Alpine Linux and they requires to be run in a flatpak environment which is something different than running them natively ...
Ex: thunderbird !

[krakjoe]

804420b#r67951874

…

On Fri, 4 Mar 2022 at 11:07, Radu Ovidiu Ilies ***@***.***> wrote: Not sure, but as far as I know Alpine linux does not use the standard libc ... Perhaps that's the problem there. — Reply to this email directly, view it on GitHub <#8160 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AARB52TG2CR67UQ6X2BRIM3U6HOGJANCNFSM5PT6OF3A> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>