From ededfc86ed8fb507db84d726f5b489bc2cdd5cd5 Mon Sep 17 00:00:00 2001 From: "Jeremy C. Reed" Date: Fri, 23 May 2014 18:06:09 +0000 Subject: [PATCH] adjust report-cpp-coverage make target exclude include/ so will remove /usr/local/include too also add a hack to change the paths to be consistent in the case that the directory the job is done in is a symlink to another directory. (Alternative method would be to run the lcov tool multiple times for each path and set the base-directory switch plus add-tracefile switch to combine.) I didn't get this reviewed. I am under assuption that this make target is rarely used. --- Makefile.am | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/Makefile.am b/Makefile.am index 6fd0200c5e..014d7aa279 100644 --- a/Makefile.am +++ b/Makefile.am @@ -78,14 +78,15 @@ report-cpp-coverage: ext/coroutine/\* \ gtest/\* \ log4cplus/\* \ - usr/include/\* \ + include/\* \ tests/\* \ unittests/\* \ \*_unittests.cc \ \*_unittest.cc \ \*_unittests.h \ --output report.info ; \ - $(GENHTML) --legend -o $(abs_top_builddir)/coverage-cpp-html report.info ; \ + sed -e "s|$(abs_top_srcdir)|$(abs_top_builddir)|g" < report.info > report.info.2 ; \ + $(GENHTML) --legend -o $(abs_top_builddir)/coverage-cpp-html report.info.2 ; \ echo "Generated C++ Code Coverage report in HTML at $(abs_top_builddir)/coverage-cpp-html" ; \ else \ echo "C++ code coverage not enabled at configuration time." ; \