Skip to content

Conversation

@crazywhalecc
Copy link
Owner

@crazywhalecc crazywhalecc commented Oct 15, 2025

What does this PR do?

Checklist before merging

If your PR involves the changes mentioned below and completed the action, please tick the corresponding option.
If a modification is not involved, please skip it directly.

  • If you modified *.php or *.json, run them locally to ensure your changes are valid:
    • composer cs-fix
    • composer analyse
    • composer test
    • bin/spc dev:sort-config
  • If it's an extension or dependency update, please ensure the following:
    • Add your test combination to src/globals/test-extensions.php.
    • If adding new or fixing bugs, add commit message containing extension test or test extensions to trigger full test suite.

@crazywhalecc
Copy link
Owner Author

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.

@crazywhalecc crazywhalecc added the wip Work In Process label Oct 20, 2025
@crazywhalecc crazywhalecc marked this pull request as draft October 20, 2025 01:26
@crazywhalecc
Copy link
Owner Author

The implementation seems to be tricky in PHP <= 8.3, which requires debugging this crash.

@crazywhalecc crazywhalecc marked this pull request as ready for review October 24, 2025 19:05
@crazywhalecc crazywhalecc removed the wip Work In Process label Oct 25, 2025
@crazywhalecc
Copy link
Owner Author

crazywhalecc commented Oct 25, 2025

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.

@henderkes
Copy link
Collaborator

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.

@crazywhalecc crazywhalecc merged commit f8c8300 into main Oct 25, 2025
9 of 10 checks passed
@crazywhalecc crazywhalecc deleted the sapi/cgi-win branch October 25, 2025 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request] Windows CGI API

3 participants