summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorElia Pinto <gitter.spiros@gmail.com>2022-04-14 02:25:13 (GMT)
committerJunio C Hamano <gitster@pobox.com>2022-04-14 05:16:36 (GMT)
commit96ddfecc5b9cd8d8b7b6316ed2523a489e879682 (patch)
treea24f9eba12607e1e400a372c401d6db90136df57
parentd516b2db0af2221bd6b13e7347abdcb5830b2829 (diff)
downloadgit-96ddfecc5b9cd8d8b7b6316ed2523a489e879682.zip
git-96ddfecc5b9cd8d8b7b6316ed2523a489e879682.tar.gz
git-96ddfecc5b9cd8d8b7b6316ed2523a489e879682.tar.bz2
Makefile: add a prerequisite to the coverage-report target
Directly invoking make coverage-report as a target results in an error because its prerequisites are missing, This patch adds the compile-test prerequisite, which is run only once each time the compile-report target is invoked. In practice, the developer may decide to review the coverage-report results without necessarily rerunning for this coverage-test, if it has already been run. Signed-off-by: Elia Pinto <gitter.spiros@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r--Makefile7
1 files changed, 6 insertions, 1 deletions
diff --git a/Makefile b/Makefile
index 5580859..72ae7fa 100644
--- a/Makefile
+++ b/Makefile
@@ -3398,6 +3398,7 @@ coverage-clean-results:
$(RM) coverage-untested-functions
$(RM) -r cover_db/
$(RM) -r cover_db_html/
+ $(RM) coverage-test.made
coverage-clean: coverage-clean-results
$(RM) $(addsuffix *.gcno,$(object_dirs))
@@ -3412,13 +3413,17 @@ coverage-compile:
coverage-test: coverage-clean-results coverage-compile
$(MAKE) CFLAGS="$(COVERAGE_CFLAGS)" LDFLAGS="$(COVERAGE_LDFLAGS)" \
DEFAULT_TEST_TARGET=test -j1 test
+ touch coverage-test.made
+
+coverage-test.made:
+ $(MAKE) coverage-test
coverage-prove: coverage-clean-results coverage-compile
$(MAKE) CFLAGS="$(COVERAGE_CFLAGS)" LDFLAGS="$(COVERAGE_LDFLAGS)" \
DEFAULT_TEST_TARGET=prove GIT_PROVE_OPTS="$(GIT_PROVE_OPTS) -j1" \
-j1 test
-coverage-report:
+coverage-report: coverage-test.made
$(QUIET_GCOV)for dir in $(object_dirs); do \
$(GCOV) $(GCOVFLAGS) --object-directory=$$dir $$dir*.c || exit; \
done