Skip to content

CI: fix build-windows by removing hardcoded VS 2022 CMake generator#847

Merged
ithewei merged 2 commits into
masterfrom
copilot/fix-build-windows-job
Jul 9, 2026
Merged

CI: fix build-windows by removing hardcoded VS 2022 CMake generator#847
ithewei merged 2 commits into
masterfrom
copilot/fix-build-windows-job

Conversation

Copilot AI commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

windows-latest now resolves to windows-2025-vs2026 (VS 2026), which no longer includes VS 2022, causing CMake to fail with Generator "Visual Studio 17 2022" could not find any instance of Visual Studio.

Change

  • Remove -G "Visual Studio 17 2022" from the cmake invocation; CMake will auto-detect the installed VS version. -A x64 is retained to preserve the x64 architecture target.
-cmake .. -G "Visual Studio 17 2022" -A x64 -DCMAKE_BUILD_TYPE=Release
+cmake .. -A x64 -DCMAKE_BUILD_TYPE=Release

Copilot AI changed the title [WIP] Fix failing GitHub Actions job build-windows CI: fix build-windows by removing hardcoded VS 2022 CMake generator Jul 9, 2026
Copilot AI requested a review from ithewei July 9, 2026 14:30
@ithewei ithewei marked this pull request as ready for review July 9, 2026 14:31
Copilot AI review requested due to automatic review settings July 9, 2026 14:31

Copilot AI 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.

Pull request overview

Updates the Windows CI job in libhv to avoid pinning a specific Visual Studio CMake generator, so the workflow keeps working as windows-latest images move to newer Visual Studio versions.

Changes:

  • Remove -G "Visual Studio 17 2022" from the Windows CMake configure step and rely on CMake’s default generator detection.
  • Keep -A x64 and existing build/test commands intact.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ithewei ithewei merged commit 2f74281 into master Jul 9, 2026
13 checks 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.

3 participants