Fix MSYS2 pacman timeout on Windows builds #975
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sunt05
force-pushed
the
sunt05/fix-msys2-download-timeout
branch
from
3013368 to
d5d6b91
Compare
…builds MSYS2 mirrors can be slow or overloaded, causing pacman to fail with "Operation too slow" errors during cibuildwheel builds on Windows runners. This fix adds: - 3-attempt retry loop with 15-second delays between attempts - --disable-download-timeout flag to prevent libcurl's speed check from triggering Resolves intermittent Windows build failures on win-cp312 (especially in UMEP variant builds). 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
When pacman upgrades core packages (pacman, msys2-runtime), MSYS2 terminates all running processes including the bash shell executing the command. This causes the command to exit with code 1 even though the upgrade succeeded. Fix by running pacman -Syu twice in separate shell invocations: - First run: ignore exit code (|| true) as shell may be terminated - Second run: complete any remaining updates in fresh shell - Package install: keep retry logic for network timeouts 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
When MSYS2 upgrades core packages, it uses taskkill to forcibly terminate all MSYS2 processes. This kills bash.exe before it can execute any error handling (|| true). The solution is to catch the error at the Windows shell level using (cmd || ver >nul), where 'ver >nul' always succeeds, allowing the command chain to continue. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
sunt05
force-pushed
the
sunt05/fix-msys2-download-timeout
branch
from
eacb895 to
07b9946
Compare
sunt05
temporarily deployed
to
github-pages-preview
GitHub Actions
Inactive
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Fixes intermittent
Operation too slowerrors from MSYS2 pacman during Windows wheel builds, particularly affecting Python 3.12 UMEP variant builds.Changes
--disable-download-timeoutflag to prevent libcurl speed checks from triggeringThis addresses the common issue where overloaded MSYS2 mirrors or GitHub runner network conditions cause pacman to fail when download speeds temporarily fall below the 1 byte/sec threshold.
Test Plan
🤖 Generated with Claude Code