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. > > Add that missing cleanup command to 't/Makefile', and all sub-Makefiles > > touched by ee65b194d as well. > > Ditto here. > > > contrib/scalar/t/Makefile | 1 + > > contrib/subtree/t/Makefile | 1 + > > t/Makefile | 1 + > > That patch itself looks sensible to me. > > -Peff
