Skip to content

Conversation

@crazywhalecc
Copy link
Owner

What does this PR do?

Add cgi support for macOS and Linux

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 crazywhalecc changed the base branch from main to feat/cgi September 4, 2025 06:10
Base automatically changed from feat/cgi to main September 5, 2025 09:13
@henderkes henderkes merged commit 8d30334 into main Sep 5, 2025
10 checks passed
@henderkes henderkes deleted the sapi/cgi branch September 5, 2025 09:52
@Lea-Reift
Copy link

But... Windows... 😢

@henderkes
Copy link
Collaborator

You can use https://windows.php.net/download/ for now. Sure it's more than just a single file, but you need to ship an extra file anyway.

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.

4 participants