Re: [PATCH 1/9] extension.partialclone: introduce partial clone extension
- From
- Jeff Hostetler <git@jeffhostetler.com>
- Date
- Nov 8, 2017, 20:32 UTC
- Message-ID
- <515130bd-6fd8-2c53-e935-e811330f8512@jeffhostetler.com>
- In-Reply-To
- <20171106111642.696a006e802cf0259098b44f@google.com>
On 11/6/2017 2:16 PM, Jonathan Tan wrote:
Show 23 quoted lines
> On Mon, 6 Nov 2017 12:32:45 -0500 > Jeff Hostetler <git@jeffhostetler.com> wrote: > >>>> Yes, that is a point I wanted to ask about. I renamed the >>>> extensions.partialclone that you created and then I moved your >>>> remote.<name>.blob-max-bytes setting to be in extensions too. >>>> Moving it to core.partialclonefilter is fine. >>> >>> OK - in that case, it might be easier to just reuse my first patch in >>> its entirety. "core.partialclonefilter" is not used until the >>> fetching/cloning part anyway. >>> >> >> Good point. I'll take a look at refactoring that. >> If it looks like the result will be mostly/effectively >> your original patches, I'll let you know and hand part 2 >> back to you. > > Sounds good. I uploaded the result of rebasing my part 2 patches on top > of your part 1 patches here, if you would like it as a reference: > > https://github.com/jonathantanmy/git/commits/pc20171106 >
Thanks Jonathan.
I moved my version of part 2 on top of yesterday's part 1. There are a few changes between my version and yours. Could you take a quick look at them and see if they make sense? (I'll spare the mailing list another patch series until after I attend to the feed back on part 1.)
https://github.com/jeffhostetler/git/commits/core/pc3_p2
Thanks Jeff