On Thu, Jan 28, 2021 at 07:10:04PM -0800, Junio C Hamano wrote: > We know that the enumerated objects do not appear in any of the kept > pack, but it does not mean all objects that are reachable/in-use that > are not in any kept packs are enumerated. You raise a very valid point. FWIW, I originally wrote these patches as just "enumerate the objects in these small packs, make a new pack out of those, and then (optionally) delete the small ones. I abandoned that idea because it needs special handling for loose objects, and it has no idea which objects are unreachable, etc. But maybe it is time to go back to the drawing board there. Perhaps a `--geometric` repack implies that we keep unreachable objects in effect, and that a full repack (i.e., one that does reachability analysis) is required to drop them. Other ideas are welcome. Thanks, Taylor
