On 11/6/2017 2:16 PM, Jonathan Tan wrote:
On Mon, 6 Nov 2017 12:32:45 -0500
Jeff Hostetler <git@xxxxxxxxxxxxxxxxx> 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
