midx-write: revert use of --stdin-packs
commit8fb6d11fade8301f0af1074fe250b9d79544cb18
authorDerrick Stolee <stolee@gmail.com>
Thu, 18 Jul 2024 19:55:46 +0000 (18 19:55 +0000)
committerJunio C Hamano <gitster@pobox.com>
Fri, 19 Jul 2024 14:19:01 +0000 (19 07:19 -0700)
tree0ce1bff1d0cba13396ddaa14e547f1718c223de2
parent738fab524ce9d666336f90034a3f600876529ffa
midx-write: revert use of --stdin-packs

This reverts b7d6f23a171 (midx-write.c: use `--stdin-packs` when
repacking, 2024-04-01) and then marks the test created in the previous
change as passing.

The fundamental issue with the reverted change is that the focus on
pack-files separates the object selection from how the multi-pack-index
selects a single pack-file for an object ID with multiple copies among
the tracked pack-files.

The change was made with the intention of improving delta compression in
the resulting pack-file, but that can be resolved with the existing
object list mechanism. There are other potential pitfalls of doing an
object walk at this time if the repository is a blobless partial clone,
and that will require additional testing on top of the one that changes
here.

Signed-off-by: Derrick Stolee <stolee@gmail.com>
Acked-by: Taylor Blau <me@ttaylorr.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
midx-write.c
t/t5319-multi-pack-index.sh