Skip to content

Conversation

@radarhere
Copy link
Member

@radarhere radarhere commented Jan 31, 2025

MinGW has started failing in main - https://github.com/python-pillow/Pillow/actions/runs/13051364081

The failure occurs with mingw-w64-x86_64-libimagequant-4.3.3-2, but did not occur with mingw-w64-x86_64-libimagequant-4.3.3-1.

MinGW does not allow installation of older versions of packages, so this PR stops installing libimagequant on MinGW for the moment.

As far as a possible long-term solution, I've created msys2/MINGW-packages#23245 to update the MinGW package to libimagequant 4.3.4, to see if that does anything.

@hugovk
Copy link
Member

hugovk commented Jan 31, 2025

As far as a possible long-term solution, I've created msys2/MINGW-packages#23245 to update the MinGW package to libimagequant 4.3.4, to see if that does anything.

Thanks, this has been merged and mingw-w64-libimagequant is waiting to be built at https://packages.msys2.org/queue to update from 4.3.3-2 to 4.3.4-1.

Let's wait a bit to see if it's working when done.

@hugovk
Copy link
Member

hugovk commented Feb 6, 2025

https://packages.msys2.org/packages/mingw-w64-x86_64-libimagequant now shows 4.3.4-1 but we're still getting:

resolving dependencies...
looking for conflicting packages...
...
Packages (83) [...] mingw-w64-x86_64-libimagequant-4.3.3-2 [...]
...
mingw-w64-x86_64-libimagequant-4.3.3-2-any downloading...

I guess one of the packages is depending on the old one?

Anyway, let's merge this to get the CI back to green, and revisit later.

Thanks!

@hugovk hugovk merged commit f55a090 into python-pillow:main Feb 6, 2025
50 of 51 checks passed
@radarhere radarhere deleted the mingw_libimagequant branch February 6, 2025 11:26
@radarhere
Copy link
Member Author

4.3.4-1 now appears in GitHub Actions, but it doesn't resolve the problem.

I've created msys2/MINGW-packages#23316 to see if anyone over there can assist.

@radarhere
Copy link
Member Author

msys2/MINGW-packages#23316 was fixed by msys2/MINGW-packages#23387, and I've now created #8778 to revert this.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants