From: Junio C Hamano Date: Thu, 12 Jun 2008 07:00:41 GMT Subject: Re: [PATCH 3/4] remote prune: print the list of pruned branches Message-ID: <7v63sf9lye.fsf@gitster.siamese.dyndns.org> In-Reply-To: <1213109495-6974-1-git-send-email-dkr+ml.git@free.fr> Olivier Marin writes: > diff --git a/builtin-remote.c b/builtin-remote.c > index 745a4ee..851bdde 100644 > --- a/builtin-remote.c > +++ b/builtin-remote.c > ... > + printf("Pruning %s\n", *argv); > + if (states.stale.nr) > + printf("From: %s\n", states.remote->url[0]); Thanks. I've queued the series (with minor fixups and rewording) to 'next' already, hoping that we can merge this fix to 'master' before 1.5.6. But I am very tempted to also apply the following on top. Thoughts? -- >8 -- [PATCH] "remote prune": be quiet when there is nothing to prune The previous commit made it always say "Pruning $remote" but reported the URL only when there is something to prune. Make it consistent by not saying anything at all when there is nothing to prune. Signed-off-by: Junio C Hamano --- builtin-remote.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/builtin-remote.c b/builtin-remote.c index 4b00cf9..145dd85 100644 --- a/builtin-remote.c +++ b/builtin-remote.c @@ -560,12 +560,13 @@ static int prune(int argc, const char **argv) get_remote_ref_states(*argv, &states, 1); - printf("Pruning %s\n", *argv); - if (states.stale.nr) + if (states.stale.nr) { + printf("Pruning %s\n", *argv); printf("URL: %s\n", states.remote->url_nr ? states.remote->url[0] : "(no URL)"); + } for (i = 0; i < states.stale.nr; i++) { const char *refname = states.stale.items[i].util; -- 1.5.6.rc2.26.g8c37