Re: [PATCH 2/2] prune.c: only print informational message in show_only or verbose mode

On Mon, Aug 6, 2012 at 10:34 PM, Junio C Hamano [off-list ref] wrote:

Junio C Hamano [off-list ref] writes:

Ok, now a tested patch, on top of your 1/2

On Mon, Aug 6, 2012 at 10:32 PM, Jeff King [off-list ref] 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