On Tue, Apr 18, 2023 at 01:12:34AM -0700, Junio C Hamano wrote: > * 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@xxxxxxxxxxxx> 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@xxxxxxxxxxxxxxxxxxxxxxx/
