make ci script params more flexible

bemanproject · Aug 24, 2024 · e411878 · e411878
1 parent f99a16f
commit e411878
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -20,7 +20,7 @@ jobs:
           submodules: recursive
       - name: Run
         run: |
-          ./ci.sh -fsanitize=address -fsanitize=undefined -fsanitize-undefined-trap-on-error
+          ./ci.sh -DCMAKE_CXX_FLAGS='-fsanitize=address -fsanitize=undefined -fsanitize-undefined-trap-on-error'
   clang-linux:
     runs-on: ubuntu-latest
     container:
@@ -32,7 +32,7 @@ jobs:
           submodules: recursive
       - name: Run
         run: |
-          ./ci.sh -fsanitize=address -fsanitize=undefined -fsanitize-undefined-trap-on-error -stdlib=libc++
+          ./ci.sh -DCMAKE_CXX_FLAGS='-fsanitize=address -fsanitize=undefined -fsanitize-undefined-trap-on-error -stdlib=libc++'
   msvc-windows:
     runs-on: windows-latest
     steps:

diff --git a/ci.sh b/ci.sh
@@ -7,7 +7,7 @@ function main() {
     local checkout="$PWD"
     mkdir "$checkout/build"
     cd "$checkout/build"
-    cmake .. -DCMAKE_BUILD_TYPE=Debug -DCMAKE_CXX_STANDARD=23 -DBUILD_TESTING=On -DCMAKE_CXX_FLAGS="${cxx_flags[*]}"
+    cmake .. -DCMAKE_BUILD_TYPE=Debug -DCMAKE_CXX_STANDARD=23 -DBUILD_TESTING=On "$@"
     cmake --build . --parallel
     ctest -C Debug
 }