fix sanitizers in CI

kelofm · Aug 17, 2023 · 13e50e3 · 13e50e3
1 parent d809d42
commit 13e50e3
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/.github/workflows/github-actions.yml b/.github/workflows/github-actions.yml
@@ -22,7 +22,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        sanitizer: [ADDRESS, THREAD, UB]
+        sanitizer: [address, thread, undefined]
     runs-on: ubuntu-latest
     steps:
       - name: Create python venv
@@ -38,10 +38,10 @@ jobs:
           sudo xargs apt install < linalg/dependencies_ubuntu.txt
       - name: Build dependencies
         # OpenMPI is leaking all over the place (according to ASAN), so disable it for now
-        #run: ./build.sh -t Debug -o "-CIEUTILS_ENABLE_${{ matrix.sanitizer }}_SANITIZER=ON" -o "-DCIEUTILS_ENABLE_MPI=ON"
-        run: source penv/bin/activate && ./cieutils/build.sh -t Debug -o "-DCIEUTILS_ENABLE_${{ matrix.sanitizer }}_SANITIZER=ON"
+        #run: ./build.sh -t Debug -o "-DCMAKE_CXX_FLAGS=-fsanitize=${{ matrix.sanitizer }}" -o "-DCIEUTILS_ENABLE_MPI=ON"
+        run: source penv/bin/activate && ./cieutils/build.sh -t Debug -o "-DCMAKE_CXX_FLAGS=-fsanitize=${{ matrix.sanitizer }}"
       - name: Build
-        run: source penv/bin/activate && ./linalg/build.sh -t Debug -o "-DLINALG_ENABLE_${{ matrix.sanitizer }}_SANITIZER=ON"
+        run: source penv/bin/activate && ./linalg/build.sh -t Debug -o "-DCMAKE_CXX_FLAGS=-fsanitize=${{ matrix.sanitizer }}"
       - name: Run tests
         run: ./linalg/build/bin/linalg_testrunner