-
-
Notifications
You must be signed in to change notification settings - Fork 334
Add Windows CGI SAPI build support #927
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The low version problem is a bit tricky. I build the exact same content locally and remotely, but both errors are reported, but the errors are different. |
|
The implementation seems to be tricky in PHP <= 8.3, which requires debugging this crash. |
# Conflicts: # src/globals/test-extensions.php
|
I've fixed the issue with older CGI builds. It's related to the Win32 API. Since the patches are more relevant to phpmicro, I've updated them to the static-php's phpmicro fork project. For PHP 8.5, it's related to some unsupported extensions, such as sqlsrv. Remi's roadmap seems to be up to date and good for reference. This isn't relevant to this pull request. We'll need to track it later in another issue or pull request. |
|
Remi's tracker is based on PR's existing or php 8.5 compatibility being merged into master. Most extensions haven't had a stable release with the fixes, so the roadmap is inapplicable unless we overhaul our source.json into an unstable state, which I do not want to do. We'll just have to wait - until then, some extensions just won't be 8.5 compatible. |
What does this PR do?
Checklist before merging
*.phpor*.json, run them locally to ensure your changes are valid:composer cs-fixcomposer analysecomposer testbin/spc dev:sort-configsrc/globals/test-extensions.php.extension testortest extensionsto trigger full test suite.