From: Brandon Casey <drafnel@gmail.com>
Date: Tue, 07 Aug 2012 05:44:07 GMT
Subject: Re: [PATCH 2/2] prune.c: only print informational message in show_only or verbose mode
Message-ID: <CA+sFfMdVhTwAFLUgrO-mLBh8apG-5X1OJKCN9xgq3-N+1RBrvg@mail.gmail.com>
In-Reply-To: <7vpq73w9i8.fsf@alter.siamese.dyndns.org>

On Mon, Aug 6, 2012 at 10:34 PM, Junio C Hamano <gitster@pobox.com> wrote:
> Junio C Hamano <gitster@pobox.com> writes:
>
> Ok, now a tested patch, on top of your 1/2


On Mon, Aug 6, 2012 at 10:32 PM, Jeff King <peff@peff.net> wrote:
>
> This seems like a band-aid. The real problem is that auto-gc can
> interfere with the pack protocol, which it should not be allowed to do,
> no matter what it produces.
>
> We could fix that root cause with this patch (on top of your 1/2):

Anyone else? :)

Ah, I wasn't aware of that feature of run_command.  Both look obviously correct.

And the comment I made yesterday about leaking the full path to the
remote end can be disregarded, since prune will report the path
relative to the repository base.

Thanks,
-Brandon