"Shawn O. Pearce" [off-list ref] writes:
Daniel Barkalow [off-list ref] wrote:
quoted
I think free_pack_by_name() also needs to drop the entries that are from
the freed pack, to avoid having repack able to get the same problem,
although I wouldn't be surprised if repack happened to never allocate a
new pack after freeing an old pack with stale delta cache entries, or
never used the delta cache after that, simply because it does one thing
and then exits.
Oy. I missed that we added this function. Patch follows.
Thanks, both.