Re: [PATCH] t/Makefile: remove 'test-results' on 'make clean'
- From
Jeff King <peff@peff.net>
- Date
- Sep 20, 2022, 20:42 UTC
- Message-ID
- <Yyol0c2op5Z2BeK1@coredump.intra.peff.net>
- In-Reply-To
- <20220920201150.GB1704@szeder.dev>
On Tue, Sep 20, 2022 at 10:11:50PM +0200, SZEDER Gábor wrote:
Show 17 quoted lines
> On Tue, Sep 20, 2022 at 03:51:14PM -0400, Jeff King wrote: > > On Tue, Sep 20, 2022 at 12:54:07PM +0200, SZEDER Gábor wrote: > > > > > The 't/test-results' directory and its contents are by-products of the > > > test process, so 'make clean' should remove them, but, alas, this has > > > been broken since ee65b194d (t/Makefile: don't remove test-results in > > > "clean-except-prove-cache", 2022-07-28). > > > > I don't have that commit. > > Uh-oh. Me neither :) > > > I assume you mean fee65b194d, and what you > > have here was a version before it hit 'next'. > > Looking at the reflog, it seems that while rewording the commit > message I inadvertently deleted the first character of the SHA1.
Heh. You'd think I would have notice the similarity when pasting in the new one. :) But yeah, that is definitely what happened.
-Peff