Skip to content

Commit

Permalink
ci: convert valgrind to cmake (#529)
Browse files Browse the repository at this point in the history
* ci: convert valgrind to cmake

* valgrind: use cmake build

* valgrind: minor fixes

* valgrind: build rem with cmake

* coverage: enable verbose retest
  • Loading branch information
alfredh authored Sep 17, 2022
1 parent e2a1525 commit 9b78d72
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 7 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ jobs:
rm re/libre.dylib || true
make -C rem librem.a
cd retest; cmake -B build -DCMAKE_EXE_LINKER_FLAGS="--coverage" && \
cmake --build build -j && ./build/retest -a
cmake --build build -j && ./build/retest -v -a
- name: gcov
run: |
Expand Down
11 changes: 5 additions & 6 deletions .github/workflows/valgrind.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@ jobs:
sudo apt-get update && sudo apt-get install -y libssl-dev valgrind
- name: make
run: make EXTRA_CFLAGS=-Werror CCACHE=
run: |
cmake -B build && cmake --build build -j
- uses: sreimers/[email protected]
with:
Expand All @@ -33,9 +34,7 @@ jobs:
- name: retest
run: |
cd ..
rm re/libre.so || true
rm re/libre.dylib || true
make -C rem librem.a
cmake -S rem -B rem/build && cmake --build rem/build
cd retest
make
valgrind --leak-check=full --show-reachable=yes --error-exitcode=42 ./retest -r
cmake -B build && cmake --build build -j
valgrind --leak-check=full --show-reachable=yes --error-exitcode=42 ./build/retest -r

0 comments on commit 9b78d72

Please sign in to comment.