Re: fact-import: failed to apply delta
"Shawn O. Pearce" <spearce@spearce.org> writes:
Show 10 quoted lines
> Daniel Barkalow <barkalow@iabervon.org> 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.