ci: enable valgrind for build

baresip · Sep 7, 2024 · 92af0ba · 92af0ba
1 parent a0c0a8d
commit 92af0ba
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -32,15 +32,10 @@ jobs:
       run: |
         echo "OPENSSL_ROOT_DIR=$(brew --prefix openssl)" >> $GITHUB_ENV
 
-    - name: fix flaky azure mirrors
-      if: ${{ runner.os == 'Linux' }}
-      run: |
-        sudo sed -i 's/azure\./de\./' /etc/apt/sources.list
-
     - name: install packages
       if: ${{ runner.os == 'Linux' }}
       run: |
-        sudo apt-get update && sudo apt-get install -y ninja-build
+        sudo apt-get update && sudo apt-get install -y ninja-build valgrind
 
     - name: install packages
       if: ${{ runner.os == 'macOS' }}
@@ -55,9 +50,15 @@ jobs:
 
     - name: cmake
       run: |
-        cmake -B build -DCMAKE_BUILD_TYPE=${{ matrix.build_type }} -Werror -DCMAKE_C_FLAGS="-Werror"
+        cmake -B build -DCMAKE_BUILD_TYPE=${{ matrix.build_type }} -DCMAKE_C_FLAGS="-Werror"
         cmake --build build -t retest
     
+    - name: retest with valgrind
+      if: ${{ runner.os == 'Linux' }}
+      run: |
+        valgrind --leak-check=full --show-reachable=yes --error-exitcode=42 ./build/test/retest -a -v
+
     - name: retest
+      if: ${{ runner.os == 'macOS' }}
       run: |
-        ./build/test/retest -r -v
+        ./build/test/retest -a -v