Skip to content

Fix win_amd64 pip CI: skip all free-threaded builds, not just cp314t - #9430

Merged
alexreinking merged 1 commit into
mainfrom
alexreinking/fix-win-freethreaded-wheel-skip
Sep 5, 2026
Merged

alexreinking merged 1 commit into
mainfrom
alexreinking/fix-win-freethreaded-wheel-skip

Conversation

@alexreinking

Copy link
Copy Markdown
Member

Summary

  • Build halide-runtime wheels for win_amd64 (and Build Halide wheels for win_amd64) started failing on 2026-08-31 after Bump pypa/cibuildwheel from 4.1.1 to 4.2.0 #9402 bumped cibuildwheel to 4.2.0, which added recognition of CPython 3.15's free-threaded cp315t-win_amd64 build target.
  • That target hits the same MSVC link failure previously seen on cp314t-win_amd64 (LINK1104: cannot open file 'python315.lib'), since CMake's FindPython/pybind11 machinery probes for the non-t import lib name that doesn't exist for free-threaded interpreters.
  • This was already fixed generically once, in Bump pypa/cibuildwheel from 4.1.1 to 4.2.0 #9298, by changing the skip pattern from the version-pinned cp314t-* to the version-agnostic cp3??t-*. However, [python] Split Python packages and add standalone AOT runtime #9272 (the halide-bin/halide/halide-runtime package split) branched before that fix landed, and its rewrite of pip.yml reintroduced the narrower cp314t-* pattern in both new jobs — silently regressing the fix until cibuildwheel 4.2.0 made cp315t buildable again.
  • This PR restores cp3??t-* in both CIBW_SKIP lines so all free-threaded Windows variants are excluded regardless of Python version.

Test plan

  • CI: confirm Build halide-runtime wheels for win_amd64 and Build Halide wheels for win_amd64 pass on this PR

🤖 Generated with Claude Code

cibuildwheel 4.2.0 (#9402) started recognizing CPython 3.15, whose
free-threaded cp315t-win_amd64 build hits the same MSVC link failure
(missing pythonXY.lib) that cp314t-win_amd64 was excluded for. The
version-agnostic `cp3??t-*` skip pattern already fixed this once
(#9298), but the halide-bin/halide/halide-runtime package split
(#9272) branched before that fix landed and reintroduced the
narrower `cp314t-*` pattern, silently regressing it.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@alexreinking
alexreinking merged commit a3690b3 into main Sep 5, 2026
1 check passed
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.

2 participants