tb/enable-cruft-packs-by-default (was: Re: What's cooking in git.git (Apr 2023, #05; Mon, 17))

From: Taylor Blau <me@ttaylorr.com>
Date: Apr 18, 2023, 14:54 UTC
Message-ID: <ZD6vQ2wro8aQ/g+y@nand.local>
In-Reply-To: <xmqqfs8xfw25.fsf@gitster.g>

On Tue, Apr 18, 2023 at 01:12:34AM -0700, Junio C Hamano wrote:

Show 13 quoted lines

> * tb/enable-cruft-packs-by-default (2023-04-17) 10 commits
>  - repository.h: drop unused `gc_cruft_packs`
>  - builtin/gc.c: make `gc.cruftPacks` enabled by default
>  - t/t6501-freshen-objects.sh: prepare for `gc --cruft` by default
>  - t/t6500-gc.sh: add additional test cases
>  - t/t6500-gc.sh: refactor cruft pack tests
>  - t/t9300-fast-import.sh: prepare for `gc --cruft` by default
>  - t/t5304-prune.sh: prepare for `gc --cruft` by default
>  - builtin/gc.c: ignore cruft packs with `--keep-largest-pack`
>  - builtin/repack.c: fix incorrect reference to '-C'
>  - pack-write.c: plug a leak in stage_tmp_packfiles()
>
>  source: <cover.1681764848.git.me@ttaylorr.com>

Peff took a look through this series in detail and made a handful of suggestions. In [1] he suggested that none of these were significant enough to warrant a reroll.

But if we're going to do it, let's do it right ;-). I'll send a (small) reroll today that should be ready to go.

Thanks, Taylor

[1]: https://lore.kernel.org/git/20230418110424.GD516715@coredump.intra.peff.net/