Skip to content

Commit 50916cf

Browse files
delta1roconnor-blockstream
authored andcommitted
simplicity: enable coverage in configure.ac
also filters src/simplicity/secp256k1 from the coverage results
1 parent 6c69253 commit 50916cf

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

Makefile.am

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -195,6 +195,7 @@ LCOV_FILTER_PATTERN = \
195195
-p "src/crypto/ctaes" \
196196
-p "src/minisketch" \
197197
-p "src/secp256k1" \
198+
-p "src/simplicity/secp256k1" \
198199
-p "depends"
199200

200201
DIR_FUZZ_SEED_CORPUS ?= qa-assets/fuzz_seed_corpus

configure.ac

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -862,7 +862,7 @@ if test "$use_lcov" = "yes"; then
862862

863863
AX_CHECK_LINK_FLAG([--coverage], [LDFLAGS="$LDFLAGS --coverage"],
864864
[AC_MSG_ERROR([lcov testing requested but --coverage linker flag does not work])])
865-
AX_CHECK_COMPILE_FLAG([--coverage],[CXXFLAGS="$CXXFLAGS --coverage"],
865+
AX_CHECK_COMPILE_FLAG([--coverage],[CXXFLAGS="$CXXFLAGS --coverage";CFLAGS="$CFLAGS --coverage"],
866866
[AC_MSG_ERROR([lcov testing requested but --coverage flag does not work])])
867867
CXXFLAGS="$CXXFLAGS -Og"
868868
fi

0 commit comments

Comments
 (0)