Olivier Marin <dkr+ml.git@free.fr> writes: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 <gitster@pobox.com> --- 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 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
| Stephen Smalley | Re: [AppArmor 39/45] AppArmor: Profile loading and manipulation, pathname matching |
| Tarkan Erimer | Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3 |
| Greg KH | [GIT PATCH] driver core patches against 2.6.24 |
| Robin Holt | Re: Linux 2.6.26-rc1 |
git: | |
| David Fenyes | sigsetmask()? (LINUX) |
| Theodore Ts'o | Re: SVGA-alphanum. modes |
| Rob Coleman | S3 |
| Ian Kluft | 2nd CFV and VOTE ACK: comp.os.linux reorganization |
