From: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
To: git@vger.kernel.org
Cc: "Junio C Hamano" <gitster@pobox.com>,
"Thomas Rast" <trast@student.ethz.ch>,
"Jonathan Nieder" <jrnieder@gmail.com>,
"Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
Subject: [PATCH v3 0/7] Detailed test coverage reports for Git
Date: Sun, 25 Jul 2010 19:52:38 +0000 [thread overview]
Message-ID: <1280087565-24671-1-git-send-email-avarab@gmail.com> (raw)
Here's v3 of the patch. It's almost entirely changes from Jonathan
Nieder mentioned in the review for v2 (minus the bugs). Thanks
Jonathan!
Here's the stat from v2:
.gitignore | 12 ++++++------
Makefile | 15 ++++++++++-----
t/README | 8 ++++----
3 files changed, 20 insertions(+), 15 deletions(-)
And the diff:
diff --git a/.gitignore b/.gitignore
index baed247..0a30a7e 100644
--- a/.gitignore
+++ b/.gitignore
@@ -183,0 +184,6 @@
+*.gcda
+*.gcno
+*.gcov
+/coverage-untested-functions
+/cover_db/
+/cover_db_html/
@@ -207,6 +212,0 @@
-*.gcda
-*.gcno
-*.gcov
-/coverage-untested-functions
-/cover_db
-/cover_db_html
diff --git a/Makefile b/Makefile
index b6975aa..fdfa4fe 100644
--- a/Makefile
+++ b/Makefile
@@ -1487,0 +1488 @@ ifndef V
+ QUIET_GCOV = @echo ' ' GCOV $@;
@@ -2287,0 +2289,3 @@ coverage-clean:
+ $(RM) coverage-untested-functions
+ $(RM) -r cover_db/
+ $(RM) -r cover_db_html/
@@ -2290,0 +2295 @@ COVERAGE_LDFLAGS = $(CFLAGS) -O0 -lgcov
+GCOVFLAGS = --preserve-paths --branch-probabilities --all-blocks
@@ -2298,2 +2303,2 @@ coverage-report:
- for dir in $(object_dirs); do \
- gcov --preserve-paths --branch-probabilities --all-blocks --object-directory=$$dir $$dir*.c; \
+ $(QUIET_GCOV)for dir in $(object_dirs); do \
+ gcov $(GCOVFLAGS) --object-directory=$$dir $$dir*.c || exit; \
@@ -2305 +2310 @@ coverage-untested-functions: coverage-report
- | tee coverage-untested-functions
+ > coverage-untested-functions
@@ -2307 +2312 @@ coverage-untested-functions: coverage-report
-coverage-report-cover-db: coverage-report
+cover_db: coverage-report
@@ -2310 +2315 @@ coverage-report-cover-db: coverage-report
-coverage-report-cover-db-html: coverage-report-cover-db
+cover_db_html: cover_db
diff --git a/t/README b/t/README
index 400e2da..4fe8d50 100644
--- a/t/README
+++ b/t/README
@@ -270 +270 @@ Do:
-
+
@@ -534 +534 @@ directory):
-
+
@@ -554 +554 @@ Devel::Cover module. To install it do:
-
+
@@ -557 +557 @@ Then, at the top-level:
- make coverage-report-cover-db-html
+ make cover_db_html
Ævar Arnfjörð Bjarmason (7):
gitignore: Ignore files generated by "make coverage"
Makefile: Include subdirectories in "make cover" reports
Makefile: Split out the untested functions target
Makefile: Add coverage-report-cover-db target
Makefile: Add coverage-report-cover-db-html target
t/README: A new section about test coverage
t/README: Add a note about the dangers of coverage chasing
.gitignore | 6 ++++++
Makefile | 24 +++++++++++++++++++++---
t/README | 51 +++++++++++++++++++++++++++++++++++++++++++++++++++
3 files changed, 78 insertions(+), 3 deletions(-)
next reply other threads:[~2010-07-25 19:53 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-25 19:52 Ævar Arnfjörð Bjarmason [this message]
2010-07-25 19:52 ` [PATCH v3 1/7] gitignore: Ignore files generated by "make coverage" Ævar Arnfjörð Bjarmason
2010-07-25 19:52 ` [PATCH v3 2/7] Makefile: Include subdirectories in "make cover" reports Ævar Arnfjörð Bjarmason
2010-07-25 19:52 ` [PATCH v3 3/7] Makefile: Split out the untested functions target Ævar Arnfjörð Bjarmason
2010-07-25 19:52 ` [PATCH v3 4/7] Makefile: Add coverage-report-cover-db target Ævar Arnfjörð Bjarmason
2010-07-25 19:52 ` [PATCH v3 5/7] Makefile: Add coverage-report-cover-db-html target Ævar Arnfjörð Bjarmason
2010-07-25 19:52 ` [PATCH v3 6/7] t/README: A new section about test coverage Ævar Arnfjörð Bjarmason
2010-07-25 19:52 ` [PATCH v3 7/7] t/README: Add a note about the dangers of coverage chasing Ævar Arnfjörð Bjarmason
2010-07-26 7:43 ` [PATCH v3 0/7] Detailed test coverage reports for Git Thomas Rast
2010-07-26 11:43 ` Ævar Arnfjörð Bjarmason
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1280087565-24671-1-git-send-email-avarab@gmail.com \
--to=avarab@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=jrnieder@gmail.com \
--cc=trast@student.ethz.ch \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).