ci: Standardize MSVC build process

Both the nightly release and manual-release builds use `msbuild`, while the CI job for PRs use `cmake`. This leads to a weird situation where both the release builds run without issue, but the CI job fails. Signed-off-by: David Li <[email protected]>
cataclysmbnteam · Jan 10, 2025 · f13f1eb · f13f1eb
1 parent bcca74f
commit f13f1eb
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 9 deletions.
diff --git a/...ub/workflows/msvc-full-features-cmake.yml → .github/workflows/msvc-full-features.yml b/...ub/workflows/msvc-full-features-cmake.yml → .github/workflows/msvc-full-features.yml
@@ -1,4 +1,4 @@
-name: Cataclysm Windows build (CMake + MSVC)
+name: Cataclysm Windows build (MSVC)
 
 on:
   push:
@@ -34,7 +34,7 @@ on:
 
 # We only care about the latest revision, so cancel previous instances.
 concurrency:
-  group: msvc-cmake-build-${{ github.ref_name }}
+  group: msvc-build-${{ github.ref_name }}
   cancel-in-progress: true
 
 env:
@@ -84,15 +84,10 @@ jobs:
       run: |
         vcpkg integrate install
 
-    - uses: ammaraskar/msvc-problem-matcher@master
-    - name: Configure
-      run: |
-          cmake -DTESTS=ON --preset $env:CMAKE_PRESET
-
     - uses: ammaraskar/msvc-problem-matcher@master
     - name: Build
       run: |
-          cmake --build out/build/$env:CMAKE_PRESET --config RelWithDebInfo
+          msbuild -m -p:Configuration=Release -p:Platform=x64 "-target:Cataclysm-vcpkg-static;JsonFormatter-vcpkg-static;Cataclysm-test-vcpkg-static" msvc-full-features\Cataclysm-vcpkg-static.sln
 
     - name: Dump logs if build failed
       if: failure()

diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -225,7 +225,7 @@ jobs:
         uses: lukka/get-cmake@latest
 
       - name: Install vcpkg
-        uses: lukka/run-vcpkg@v11
+        uses: lukka/run-vcpkg@main
         id: runvcpkg
         with:
           vcpkgDirectory: '${{ runner.workspace }}/b/vcpkg'