Re: [PATCH] fast-import: Stream very large blobs directly to pack

From: Junio C Hamano <gitster@pobox.com>
Date: Jan 30, 2010, 03:41 UTC
Message-ID: <7vockc45ut.fsf@alter.siamese.dyndns.org>
In-Reply-To: <20100129183705.GB22101@spearce.org>

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

Show 11 quoted lines

> Sverre Rabbelier <srabbelier@gmail.com> wrote:
>> On Fri, Jan 29, 2010 at 02:23, Shawn O. Pearce <spearce@spearce.org> wrote:
>> > index 60d0aa2..8114f93 100644
>> 
>> Looks like you based it off an old version of fast-import, at least
>> not on what is in master atm, since that has my fast-import series.
>> Rebasing shouldn't be too hard, you'll just have to move the option
>> parsing to the appropriate function.
>
> I was intentionally slating this for maint, to fix a bug a user
> reported when handling large streams.

I personally see that as adding a new feature (especially with new option and config).

> But yea... I guess I also owe Junio a rebased form for master so
> he has less merge conflicts to resolve.

Thanks.