-
-
Notifications
You must be signed in to change notification settings - Fork 7
Open
Labels
ciCI/CD, GitHub Actions, testing infrastructureCI/CD, GitHub Actions, testing infrastructurestatic-buildStatic compilation, FrankenPHP, static-php-cliStatic compilation, FrankenPHP, static-php-cli
Description
FrankenPHP requires ZTS (Zend Thread Safety). We need to verify php-debugger works under ZTS.
What to do
- Add
--enable-ztsbuild variants totests.yml(at least PHP 8.4 × ZTS) - Fix any thread-safety issues that surface (missing TSRM macros, non-thread-local globals)
- Verify all TSRM cache macros are present:
ZEND_TSRMLS_CACHE_DEFINE(),ZEND_TSRMLS_CACHE_UPDATE()
Acceptance criteria
- CI runs tests with
--enable-ztson at least one PHP version - All tests that pass in NTS also pass in ZTS
- No thread-safety warnings/errors
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
ciCI/CD, GitHub Actions, testing infrastructureCI/CD, GitHub Actions, testing infrastructurestatic-buildStatic compilation, FrankenPHP, static-php-cliStatic compilation, FrankenPHP, static-php-cli