Skip to content

CI: Fix Windows MSVC pip build on VS 2026 runner#1889

Merged
ax3l merged 1 commit into
openPMD:devfrom
ax3l:fix-windows-ci-vs2026
Jun 22, 2026
Merged

CI: Fix Windows MSVC pip build on VS 2026 runner#1889
ax3l merged 1 commit into
openPMD:devfrom
ax3l:fix-windows-ci-vs2026

Conversation

@ax3l

@ax3l ax3l commented Jun 22, 2026

Copy link
Copy Markdown
Member

The windows-latest runner now ships Visual Studio 2026 (VS 18). In the "MSVC w/o MPI via pip" job, the pip-provided CMake (4.3.x) defaults to the "NMake Makefiles" generator, which rejects setup.py's -A x64 platform argument:

Generator NMake Makefiles does not support platform specification,
but platform x64 was specified.
CMAKE_C_COMPILER not set, after EnableLanguage

CMake 4 was rejected in the installer due to an upper cap in pyproject.toml

The "MSVC w/o MPI via pip" job failed on the windows-latest runner after
it upgraded to Visual Studio 2026 (VS 18):

    Generator NMake Makefiles does not support platform specification,
    but platform x64 was specified.
    CMAKE_C_COMPILER not set, after EnableLanguage

`pip wheel` builds in an isolated environment that installs CMake per
pyproject.toml's build requirement. The `<4.0.0` cap pinned that CMake to
3.x, which predates the "Visual Studio 18 2026" generator (added in CMake
4.2), so CMake fell back to "NMake Makefiles" and rejected setup.py's
`-A x64`.

Drop the CMake upper bound so the isolated build uses a current CMake that
auto-detects the VS 2026 generator, matching the runner's own CMake that
the non-pip MSVC job already builds with. Also bump the job to Python 3.11.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ax3l ax3l force-pushed the fix-windows-ci-vs2026 branch from 35d0ac1 to 33b96a2 Compare June 22, 2026 21:38
@ax3l ax3l merged commit 2d11893 into openPMD:dev Jun 22, 2026
30 of 31 checks passed
@ax3l ax3l deleted the fix-windows-ci-vs2026 branch June 22, 2026 22:02
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.

2 participants