On Fri, Mar 7, 2014 at 1:37 AM, Junio C Hamano <gits...@pobox.com> wrote:
> I like what I see in this patch, but I wonder if we can essentially
> revert that "temporary shallow file" patch and replace it with the
> same (or a similar) mechanism uniformly?
Using --shallow-file is uniform.The only downside is temporary file.
Without it you'll need to think of a way (probably different way for
each command) to feed shallow info to.
> On the receive-pack side, the comment at the bottom of
> preprare_shallow_update() makes it clear that, if we wanted to use
> hooks, we cannot avoid having the proposed new shallow-file in a
> temporary file, which is unfortunate. Do we have a similar issue on
> hooks on the upload-pack side?
No. I don't think we have hooks on upload-pack.
>> builtin/pack-objects.c | 7 +++++++
>> shallow.c | 2 ++
>> t/t5537-fetch-shallow.sh | 13 +++++++++++++
>> upload-pack.c | 21 ++++++++++++---------
>> 4 files changed, 34 insertions(+), 9 deletions(-)
> Nothing for Documentation/ anywhere?
Heh git-pack-objects.txt never described stdin format. At least I
searched for --not in it and found none. So I gladly accepted the
situation and skipped doc update :D
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html