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

[RFC/PATCH] t/perf: "make clean" from the top-level to clean results

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 28, 2016, 20:16 UTC
Message-ID
<xmqqfuvaxsr1.fsf@gitster.mtv.corp.google.com>

Running "make clean" from the top-level after running perf tests left t/perf/test-results/ directory and tons of files in it. At least "make distclean" should turn things back to pristine state.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
---
 * Perhaps I am missing some reason why this was deliberately left
   out when we added t/perf/Makefile that does have the clean
   target?  Cc'ing the suspects found by "shortlog t/perf".
 t/Makefile | 1 +
 1 file changed, 1 insertion(+)
diff --git a/t/Makefile b/t/Makefile
index 18e2b28..e0aef9a 100644
--- a/t/Makefile
+++ b/t/Makefile
@@ -48,6 +48,7 @@ pre-clean:
 clean-except-prove-cache:
 	$(RM) -r 'trash directory'.* '$(TEST_RESULTS_DIRECTORY_SQ)'
 	$(RM) -r valgrind/bin
+	cd perf && $(MAKE) clean
 
 clean: clean-except-prove-cache
 	$(RM) .prove
Next: Jeff King
Message 1 of 3 in “t/perf: "make clean" from the top-level to clean results”
  1. t/perf: "make clean" from the top-level to clean resultsJunio C Hamano, Mar 28, 2016
  2. Jeff KingMar 28, 2016
  3. Beat BolliMar 29, 2016

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.