[PATCH v2 0/2] pack-objects: more brown-paper-bag multi-pack reuse fixes
- From
Taylor Blau <me@ttaylorr.com>
- Date
- Nov 14, 2024, 13:42 UTC
- Message-ID
- <cover.1731591708.git.me@ttaylorr.com>
- In-Reply-To
- <cover.1731518931.git.me@ttaylorr.com>
This is a small re-roll of my series to fix a regression in multi-pack reuse based on a broken assumption in write_reused_pack_verbatim().
The only change since last time is removing a stray debugging aid, and the series is otherwise identical as in v1. A range-diff is included below for convenience.
Thanks in advance for your review!
Taylor Blau (2): t5332-multi-pack-reuse.sh: demonstrate duplicate packing failure pack-objects: only perform verbatim reuse on the preferred pack
builtin/pack-objects.c | 101 +++++++++++++++--------------------- t/t5332-multi-pack-reuse.sh | 22 ++++++++ 2 files changed, 65 insertions(+), 58 deletions(-)
Range-diff against v1:
1: 7a69cf84ae5 ! 1: d791b7b20c9 t5332-multi-pack-reuse.sh: demonstrate duplicate packing failure
@@ t/t5332-multi-pack-reuse.sh: test_expect_success 'duplicate objects' '
+
+ # ...and create a separate pack containing just that object
+ p="$(git pack-objects $packdir/pack <in)" &&
-+ git show-index <$packdir/pack-$p.idx &&
+
+ git multi-pack-index write --bitmap --preferred-pack=pack-$p.idx &&
+
2: 2520abf24a8 = 2: d73b8fe2d63 pack-objects: only perform verbatim reuse on the preferred packbase-commit: 25b0f41288718625b18495de23cc066394c09a92
-- 2.46.0.421.g159f2d50e75