Re: [PATCH 1/9] extension.partialclone: introduce partial clone extension
- From
- Jonathan Tan <jonathantanmy@google.com>
- Date
- Nov 6, 2017, 19:16 UTC
- Message-ID
- <20171106111642.696a006e802cf0259098b44f@google.com>
- In-Reply-To
- <99b7c61b-f8ab-43a0-0707-62ac1db4d080@jeffhostetler.com>
On Mon, 6 Nov 2017 12:32:45 -0500 Jeff Hostetler <git@jeffhostetler.com> wrote:
Show 14 quoted lines
> >> 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