From: Junio C Hamano <gitster@pobox.com>
Date: Wed, 11 Feb 2009 18:30:25 GMT
Subject: Re: fact-import: failed to apply delta
Message-ID: <7vwsbweshq.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <20090211181530.GO30949@spearce.org>

"Shawn O. Pearce" <spearce@spearce.org> writes:

> 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.

Thanks, both.