[cicd] simplify sanitizer pipeline set up

FrancoisCarouge · Oct 17, 2023 · 09233a8 · 09233a8
1 parent cdf13d2
commit 09233a8
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/.github/workflows/sanitizer.yml b/.github/workflows/sanitizer.yml
@@ -19,7 +19,8 @@ jobs:
         sanitizer:
         - name: 'Address'
           cpp_flags: '-g -O0 -fsanitize=address'
-          run_flags: 'ASAN_OPTIONS=verbosity=2:strict_string_checks=1:detect_stack_use_after_return=1:check_initialization_order=1:strict_init_order=1'
+          env:
+            ASAN_OPTIONS: 'verbosity=2:strict_string_checks=1:detect_stack_use_after_return=1:check_initialization_order=1:strict_init_order=1'
         - name: 'Leak'
           cpp_flags: '-g -O0 -fsanitize=leak'
           run_flags: 'LSAN_OPTIONS=verbosity=2'
@@ -32,7 +33,9 @@ jobs:
     name: '${{ matrix.sanitizer.name }}'
     runs-on: ubuntu-latest
     env:
+      CXX: 'g++-13'
       CXXFLAGS: '${{ matrix.sanitizer.cpp_flags }}'
+      CC: 'gcc-13'
     steps:
     - name: Harden Runner
       uses: step-security/harden-runner@1b05615854632b887b69ae1be8cbefe72d3ae423 # v2.6.0
@@ -47,9 +50,6 @@ jobs:
         sudo apt upgrade
         sudo apt install g++-13
     - name: 'Configure'
-      env:
-        CXX: 'g++-13'
-        CC: 'gcc-13'
       run: cmake -S . -B 'build'
     - name: 'Build'
       run: cmake --build 'build' --verbose --parallel 4