Fix ASan issue on Windows workflow

jimmy-park · Jan 19, 2024 · 4a71284 · 4a71284
1 parent 318e6ea
commit 4a71284
Show file tree

Hide file tree

Showing 3 changed files with 49 additions and 6 deletions.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -8,23 +8,23 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        preset: [linux-clang, linux-gcc, macos, windows]
+        preset: [linux-clang, linux-gcc, macos, windows-ninja]
         config: [Debug, Release]
         abstract: [OFF, ON]
         sanitize: [address, thread, undefined, leak, memory]
         exclude:
           - { preset: linux-gcc, sanitize: memory }
           - { preset: macos, sanitize: leak }
           - { preset: macos, sanitize: memory }
-          - { preset: windows, sanitize: thread }
-          - { preset: windows, sanitize: undefined }
-          - { preset: windows, sanitize: leak }
-          - { preset: windows, sanitize: memory }
+          - { preset: windows-ninja, sanitize: thread }
+          - { preset: windows-ninja, sanitize: undefined }
+          - { preset: windows-ninja, sanitize: leak }
+          - { preset: windows-ninja, sanitize: memory }
         include:
           - { preset: linux-clang, os: ubuntu-latest }
           - { preset: linux-gcc, os: ubuntu-latest }
           - { preset: macos, os: macos-latest }
-          - { preset: windows, os: windows-latest }
+          - { preset: windows-ninja, os: windows-latest }
     runs-on: ${{ matrix.os }}
     steps:
       - uses: actions/checkout@v4
@@ -34,6 +34,17 @@ jobs:
         run: |
           sudo apt-get update
           sudo apt-get install -y ninja-build
+      - name: Install dependencies on Windows
+        if: runner.os == 'Windows'
+        run: |
+          choco install ninja
+
+      - name: Choose the latest MSVC toolset
+        if: runner.os == 'Windows'
+        uses: TheMrMilchmann/setup-msvc-dev@v3
+        with:
+          arch: x64
+          toolset: 14.38
 
       - name: Configure
         run: >

diff --git a/cmake/preset/base.json b/cmake/preset/base.json
@@ -188,6 +188,14 @@
                 "CMAKE_C_COMPILER": "gcc",
                 "CMAKE_CXX_COMPILER": "g++"
             }
+        },
+        {
+            "name": "msvc",
+            "hidden": true,
+            "cacheVariables": {
+                "CMAKE_C_COMPILER": "cl",
+                "CMAKE_CXX_COMPILER": "cl"
+            }
         }
     ],
     "buildPresets": [

diff --git a/cmake/preset/windows.json b/cmake/preset/windows.json
@@ -13,6 +13,15 @@
                 "default"
             ]
         },
+        {
+            "name": "windows-ninja",
+            "inherits": [
+                "host-windows",
+                "target-windows",
+                "ninja",
+                "default"
+            ]
+        },
         {
             "name": "windows-arm",
             "inherits": [
@@ -58,6 +67,13 @@
                 "host-windows"
             ]
         },
+        {
+            "name": "windows-ninja",
+            "configurePreset": "windows-ninja",
+            "inherits": [
+                "host-windows"
+            ]
+        },
         {
             "name": "windows-arm",
             "configurePreset": "windows-arm",
@@ -96,6 +112,14 @@
                 "options"
             ]
         },
+        {
+            "name": "windows-ninja",
+            "configurePreset": "windows-ninja",
+            "inherits": [
+                "host-windows",
+                "options"
+            ]
+        },
         {
             "name": "windows-arm",
             "configurePreset": "windows-arm",