valgrind on fedora

baresip · Sep 7, 2024 · 4fe04ff · 4fe04ff
1 parent 2c31e9e
commit 4fe04ff
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -54,11 +54,10 @@ jobs:
         cmake --build build -t retest
     
     - name: retest
-      if: ${{ matrix.compiler == 'clang' }}
       run: |
-        ./build/test/retest -a -v
+        ./build/test/retest -r -v
 
     - name: retest with valgrind
-      if: ${{ matrix.compiler == 'gcc' }}
+      if: ${{ matrix.os == 'ubuntu-20.04' }}
       run: |
-        valgrind --leak-check=full --show-reachable=yes --error-exitcode=42 ./build/test/retest -a -v
+        valgrind --leak-check=full --show-reachable=yes --error-exitcode=42 ./build/test/retest -r -v
diff --git a/.github/workflows/fedora.yml b/.github/workflows/fedora.yml
@@ -26,7 +26,7 @@ jobs:
     - uses: actions/checkout@v4
     - name: install devel tools
       run: |
-        yum -y install gcc clang cmake make openssl-devel zlib-devel ninja-build
+        yum -y install gcc clang cmake valgrind openssl-devel zlib-devel ninja-build
 
     - name: make info
       run: |
@@ -38,3 +38,7 @@ jobs:
     - name: make
       run: |
         cmake -B build -DCMAKE_C_FLAGS="-Werror" && cmake --build build -j
+
+    - name: retest
+      run: |
+        valgrind --leak-check=full --show-reachable=yes --error-exitcode=42 ./build/test/retest -r -v