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

3 messages from 2016-03-28 to 2016-03-29. Participants: Junio C Hamano, Jeff King, Beat Bolli.
Thread: https://gitlist.dev/t/41847

## Junio C Hamano, 2016-03-28 20:16

Subject: [RFC/PATCH] t/perf: "make clean" from the top-level to clean results
Message-ID: <xmqqfuvaxsr1.fsf@gitster.mtv.corp.google.com>
URL: https://gitlist.dev/e/xmqqfuvaxsr1.fsf%40gitster.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

```

## Jeff King, 2016-03-28 20:20

Subject: Re: [RFC/PATCH] t/perf: "make clean" from the top-level to clean results
Message-ID: <20160328202042.GA23792@sigill.intra.peff.net>
URL: https://gitlist.dev/e/20160328202042.GA23792%40sigill.intra.peff.net
In-Reply-To: <xmqqfuvaxsr1.fsf@gitster.mtv.corp.google.com>

```
On Mon, Mar 28, 2016 at 01:16:50PM -0700, Junio C Hamano wrote:

> 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".

I don't think I've ever touched the "clean" code path.

This change is fine by me. I have noticed that the contents of
t/perf/build can pile up and consume quite a lot of space, as they are
all full builds of git. They're a little more expensive to reproduce
than some other things, but they're inherently still a cache. I think
your patch is doing the right thing.

-Peff

```

## Beat Bolli, 2016-03-29 10:44

Subject: Re: [RFC/PATCH] t/perf:
Message-ID: <loom.20160329T124227-493@post.gmane.org>
URL: https://gitlist.dev/e/loom.20160329T124227-493%40post.gmane.org
In-Reply-To: <xmqqfuvaxsr1.fsf@gitster.mtv.corp.google.com>

```
Junio C Hamano <gitster <at> pobox.com> writes:

> +	cd perf && $(MAKE) clean

I think the clean way to do this is

	$(make) -C perf clean

Regards,
Beat

```