Fix PHP 8.4 builds on Windows#131
Fix PHP 8.4 builds on Windows#131simonhamp merged 7 commits intoNativePHP:mainfrom SRWieZ:fix-windows-build
Conversation
|
Research is complete and ready to be merged. During the build, there is a linking issue with A patch from php-micro is applied when building the CLI version, but it is outdated with the recent changes in PHP 8.4. Once this is resolved, everything should work smoothly. I suggest merging my PR to address the current unrelated problems and waiting for this fix to be implemented. Have a good day! |
|
I did something silly. While taking a shower, I had an idea that I didn't think would actually work, but it did. I had to learn about the .patch syntax—never thought I would! 😄 In the end, Windows 8.4 builds work! 🎉 See the build and the PR. Is it stupid if it works? Afterward, I did some research, and the original patch for the Windows CLI is based on this file:
I can safely assume that this line is no longer needed and that we don't require new code here either. I will submit a pull request to the appropriate repository later. |
simonhamp
left a comment
There was a problem hiding this comment.
While I'd definitely prefer that these patches happen upstream, I think for now this is fine.
It just feels like we'd have a lot more to manage long-term if we try to take care of this here and it could make our GH Actions kinda flakey... so I'd avoid adding further patch logic here
|
Of course! I have already created a PR upstream: static-php/phpmicro#5. The minute it's merged, I will propose another PR to clean this up. |
|
Ouchy, looks like this broke some builds: https://github.com/NativePHP/php-bin/actions/runs/12531270993 |
|
Yeah I see why, sed doesn't work the same on unix and windows. PR incoming. |
|
Here: #133 |
Fix the following:
Unfortunately, this does not resolve the PHP 8.4 Windows build issue, but it does fix and improve the current build action.
The last remaining error is related to the following patch that @mpociot commented on: static-php/phpmicro#3 (comment)
If you're curious about my research that led to those commits, see https://github.com/SRWieZ/fork-nativephp-php-bin/pull/1.