Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Switch to windows-2022 in CI #18927

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

Merged
merged 2 commits into from
Jun 24, 2025
Merged

Conversation

shivammathur
Copy link
Member

windows-2019 runner will be dropped by GitHub on 2025-06-30.

Ref: actions/runner-images#12045

Copy link
Member

@iluuu1994 iluuu1994 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @shivammathur!

windows-2019 runner will be dropped by GitHub on 2025-06-30.
Copy link
Member

@cmb69 cmb69 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @shivammathur!

I'm generally fine with the PR, but if you prefer to rewrite find-vs-toolset.bat in Powershell, I'm certainly not against that.

Regarding the test failures: almost certainly due to stack unwinding on Windows 2022; @dstogov fixed that with ccc6c0f for PHP-8.4. I don't think that we can backport that, so it might be best to mark the test as XFAIL.

@shivammathur
Copy link
Member Author

@cmb69 I think we can keep it in batch for consistency.

@cmb69
Copy link
Member

cmb69 commented Jun 24, 2025

I think we can keep it in batch for consistency.

Yeah, sure! (Actually, I'm neither happy with complex batch scripts, nor with Powershell scripts.)

@shivammathur shivammathur merged commit 6233dc6 into php:PHP-8.1 Jun 24, 2025
6 checks passed
shivammathur added a commit to shivammathur/php-src that referenced this pull request Jun 24, 2025
This is a continuation of phpGH-18927 to fix CI for windows-2022
shivammathur added a commit to shivammathur/php-src that referenced this pull request Jun 24, 2025
This is a continuation of phpGH-18927 to fix CI for windows-2022
shivammathur added a commit to shivammathur/php-src that referenced this pull request Jun 24, 2025
This is a continuation of phpGH-18927 to fix CI for windows-2022
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.

3 participants