git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v2 1/7] gitignore: Ignore files generated by "make coverage"

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Jul 25, 2010, 16:37 UTC
Message-ID
<20100725163742.GA25658@burratino>
In-Reply-To
<1280068861-17701-2-git-send-email-avarab@gmail.com>
Hi,
Ævar Arnfjörð Bjarmason wrote:
> Change gitignore to ignore the *.gcda, *.gcno and *.gcov files
> generated by GCC and our coverage invocations.
Good idea.  The following is nitpicking of the worst kind; sorry.

It is easier to maintain frequently-changing lists like .gitignore and the #include lines in source files if new additions go in some logical place in the middle instead of the end. There is less lock contention that way. :)

So maybe:
diff --git i/.gitignore w/.gitignore
index 0e61845..57f79ef 100644
--- i/.gitignore
+++ w/.gitignore
@@ -181,6 +181,10 @@
 *.[aos]
 *.py[co]
 .depend/
+*.gcda
+*.gcno
+*.gcov
+/coverage-untested-functions
 *+
 /config.mak
 /autom4te.cache
@@ -204,7 +208,3 @@
 *.pdb
 /Debug/
 /Release/
-*.gcda
-*.gcno
-*.gcov
-/coverage-untested-functions
Previous: Ævar Arnfjörð BjarmasonNext: Ævar Arnfjörð Bjarmason
Message 3 of 17 in “Detailed test coverage reports for Git”
  1. 0/7 Detailed test coverage reports for GitÆvar Arnfjörð Bjarmason, Jul 25, 2010
  2. 1/7 gitignore: Ignore files generated by "make coverage"Ævar Arnfjörð Bjarmason, Jul 25, 2010
  3. Jonathan NiederJul 25, 2010
  4. 2/7 Makefile: Include subdirectories in "make cover" reportsÆvar Arnfjörð Bjarmason, Jul 25, 2010
  5. Jonathan NiederJul 25, 2010
  6. 3/7 Makefile: Split out the untested functions targetÆvar Arnfjörð Bjarmason, Jul 25, 2010
  7. 4/7 Makefile: Add coverage-report-cover-db targetÆvar Arnfjörð Bjarmason, Jul 25, 2010
  8. Jonathan NiederJul 25, 2010
  9. 5/7 Makefile: Add coverage-report-cover-db-html targetÆvar Arnfjörð Bjarmason, Jul 25, 2010
  10. Jonathan NiederJul 25, 2010
  11. 6/7 t/README: A new section about test coverageÆvar Arnfjörð Bjarmason, Jul 25, 2010
  12. 7/7 t/README: Add a note about the dangers of coverage chasingÆvar Arnfjörð Bjarmason, Jul 25, 2010
  13. Jonathan NiederJul 25, 2010
  14. Ævar Arnfjörð BjarmasonJul 25, 2010
  15. Jonathan NiederJul 25, 2010
  16. Ævar Arnfjörð BjarmasonJul 25, 2010
  17. Jonathan NiederJul 25, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.