"Shawn O. Pearce" <spearce@xxxxxxxxxxx> writes: > Daniel Barkalow <barkalow@xxxxxxxxxxxx> wrote: >> >> 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. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html