Skip to content

Commit 9baa4f2

Browse files
committed
- add Valgrind target to Makefile and to Github Build action
Signed-off-by: Hans Zandbelt <[email protected]>
1 parent cbf6a66 commit 9baa4f2

File tree

4 files changed

+14
-0
lines changed

4 files changed

+14
-0
lines changed

.github/workflows/build.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ jobs:
1212
sudo apt-get update -y
1313
sudo apt-get install -y apache2-dev libcjose-dev libssl-dev check pkg-config
1414
sudo apt-get install -y libjansson-dev libcurl4-openssl-dev libhiredis-dev libpcre2-dev libjq-dev check
15+
sudo apt-get install -y valgrind
1516
- name: Configure
1617
run: |
1718
./autogen.sh
@@ -20,5 +21,7 @@ jobs:
2021
run: make
2122
- name: Test
2223
run: make check || (cat test-suite.log && exit -1)
24+
- name: Valgrind
25+
run: make valgrind
2326
- name: Distcheck
2427
run: make distcheck DESTDIR=/tmp/mod_auth_openidc

ChangeLog

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,6 @@
1+
08/21/2025
2+
- add Valgrind target to Makefile and to Github Build action
3+
14
08/20/2025
25
- revise test/check and code coverage functions
36
- revise autoconf/automake, split over subdirs now

Makefile.am

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,3 +15,6 @@ clang-format:
1515

1616
check-code-coverage: check
1717
${MAKE} -C test $@
18+
19+
valgrind: check
20+
${MAKE} -C test $@

test/Makefile.am

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,11 @@ EXTRA_DIST = \
7272
clean-local:
7373
rm -f *.gcno
7474

75+
${TESTS:%=%.valgrind}: ${TESTS}
76+
CK_FORK=no valgrind --error-exitcode=1 --show-leak-kinds=definite --read-inline-info=yes --keep-debuginfo=yes ./$$(basename $@ .valgrind)
77+
78+
valgrind: ${TESTS:%=%.valgrind}
79+
7580
CODE_COVERAGE_LCOV_SHOPTS = --ignore-errors inconsistent --ignore-errors unused
7681

7782
@CODE_COVERAGE_RULES@

0 commit comments

Comments
 (0)