Skip to content

Commit 13ecfa6

Browse files
committed
coverage: Pass coverage flags in CXXFLAGS
Older versions of setuptools pass CFLAGS to the C++ compiler. Newer versions of setuptools pass only CXXFLAGS to the C++ compiler. Set the profiling compiler flags in both CFLAGS and CXXFLAGS for now, to cope with both old and new versions of setuptools. Signed-off-by: Matt Wozniski <godlygeek@gmail.com>
1 parent f0e4280 commit 13ecfa6

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ valgrind: ## Run valgrind, with the correct configuration
7171
.PHONY: ccoverage
7272
ccoverage: ## Run the test suite, with C++ code coverage
7373
$(MAKE) clean
74-
CFLAGS="$(CFLAGS) -O0 -pg --coverage" $(MAKE) build
74+
CFLAGS="$(CFLAGS) -O0 -pg --coverage" CXXFLAGS="$(CXXFLAGS) -O0 -pg --coverage" $(MAKE) build
7575
$(MAKE) check
7676
gcov -i build/*/src/pystack/_pystack -i -d
7777
lcov --capture --directory . --output-file cppcoverage.lcov

0 commit comments

Comments
 (0)