On Wed, 8 Nov 2017 15:32:21 -0500 Jeff Hostetler <git@xxxxxxxxxxxxxxxxx> wrote: > 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 - the differences are quite minor, and they generally make sense. The main one is that finish_object() in builtin/rev-list.c now returns int instead of void, but that makes sense. Other than that: - I think you accidentally squashed the rev-list commit into "sha1_file: support lazily fetching missing objects". - The documentation for --exclude-promisor-objects in git-pack-objects.txt should be "Omit objects that are known to be in the promisor remote". (This option has the purpose of operating only on locally created objects, so that when we repack, we still maintain a distinction between locally created objects [without .promisor] and objects from the promisor remote [with .promisor].) - The transport options in gitremote-helpers.txt could have the same documentation as in transport.h.
