Skip to content

Test UCRT64 instead of MINGW64 - #10300

Merged
radarhere merged 6 commits into
python-pillow:mainfrom
radarhere:mingw
Oct 24, 2026
Merged

radarhere merged 6 commits into
python-pillow:mainfrom
radarhere:mingw

Conversation

@radarhere

@radarhere radarhere commented Oct 24, 2026 •

Copy link
Copy Markdown
Member

The MinGW CI job has been failing lately - https://github.com/python-pillow/Pillow/actions/runs/40053103384/job/120749911910

error: target not found: mingw-w64-x86_64-python-pyqt6

#10299 suggested switching from MINGW64 to UCRT64. Taking a look at https://www.msys2.org/docs/environments/, I found that MINGW64 is now listed as a 'Legacy Environment', and 'If you are unsure, go with UCRT64'. So this PR switches to that, in order to replicate current users' experiences better.

I have taken some inspiration from #6888

@Muguett-DBY Muguett-DBY left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One check-name mismatch in the docs-only path.

Comment thread .github/workflows/test-skipped.yml
Co-authored-by: Caldera <85372096+Muguett-DBY@users.noreply.github.com>

@hugovk hugovk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

Comment thread .github/workflows/test-msys2.yml Outdated
Comment thread .github/workflows/test-skipped.yml Outdated
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
Comment thread .github/workflows/test-msys2.yml Outdated
@radarhere
radarhere merged commit 923a595 into python-pillow:main Oct 24, 2026
53 of 54 checks passed
@hugovk

hugovk commented Oct 24, 2026

Copy link
Copy Markdown
Member

python-pillow/Pillow/settings/rules/7135275 > Require status checks to pass > Required status checks > remove the old, add the new

I've replaced removed "MinGW" with "MSYS2".

@radarhere

Copy link
Copy Markdown
Member Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants