remote show/prune: strange -n(--dry-run) option.

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Olivier Marin
Date: Saturday, June 7, 2008 - 5:54 pm

Hello,

The git-remote documentation talks about a mysterious -n option for show
and prune that comes from the old git-remote.perl script. This flag was
used to prevent the script from calling ls-remote more than once, FWIU.
Today, the builtin accept an (un)?related -n(--dry-run) flag that does
nothing, actually. It seems broken.

So, is it safe to drop it entirely or is it better to just remove it
from the documentation for compatibility? In the second case, how long
should we wait before using --dry-run for something different?

I would like to make "git remote prune" more verbose and use --dry-run
to really prevent it from deleting stale tracking branches.

$ git remote prune origin -n
Pruning origin
From: git://.../myproject.git
  * [stale branch]    bla
  * [stale branch]    bli
  * [stale branch]    blu

What about something like that ?

Olivier.
--
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
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
remote show/prune: strange -n(--dry-run) option., Olivier Marin, (Sat Jun 7, 5:54 pm)
[PATCH] remote show: fix the -n option, Olivier Marin, (Sun Jun 8, 5:48 pm)
Re: [PATCH] remote show: fix the -n option, Johannes Schindelin, (Sun Jun 8, 6:16 pm)
Re: [PATCH] remote show: fix the -n option, Olivier Marin, (Sun Jun 8, 7:06 pm)
Re: [PATCH] remote show: fix the -n option, Johannes Schindelin, (Sun Jun 8, 7:35 pm)
Re: [PATCH] remote show: fix the -n option, Olivier Marin, (Sun Jun 8, 9:16 pm)
Re: [PATCH] remote show: fix the -n option, Johannes Schindelin, (Sun Jun 8, 9:53 pm)
Re: [PATCH] remote show: fix the -n option, Olivier Marin, (Mon Jun 9, 7:22 am)
Re: [PATCH] remote show: fix the -n option, Olivier Marin, (Mon Jun 9, 8:43 am)
[PATCH v2] remote show: fix the -n option, Olivier Marin, (Mon Jun 9, 8:58 am)
Re: [PATCH] remote show: fix the -n option, Johannes Schindelin, (Mon Jun 9, 9:31 am)
Re: [PATCH v2] remote show: fix the -n option, Johannes Schindelin, (Mon Jun 9, 9:35 am)
Re: [PATCH v2] remote show: fix the -n option, Olivier Marin, (Mon Jun 9, 9:58 am)
Re: [PATCH v2] remote show: fix the -n option, Johannes Schindelin, (Mon Jun 9, 10:56 am)
Re: [PATCH v2] remote show: fix the -n option, Olivier Marin, (Mon Jun 9, 11:37 am)
[PATCH] builtin-remote: make reuse of code easier by not d ..., Johannes Schindelin, (Mon Jun 9, 1:11 pm)
Re: [PATCH v2] remote show: fix the -n option, Junio C Hamano, (Mon Jun 9, 6:10 pm)
Re: [PATCH v2] remote show: fix the -n option, Shawn O. Pearce, (Mon Jun 9, 6:19 pm)
Re: [PATCH v2] remote show: fix the -n option, Johannes Schindelin, (Mon Jun 9, 7:39 pm)
[PATCH 0/4] remote show/prune improvement, Olivier Marin, (Tue Jun 10, 7:50 am)
[PATCH 1/4] remote show: fix the -n option, Olivier Marin, (Tue Jun 10, 7:51 am)
Re: [PATCH 0/4] remote show/prune improvement, Jakub Narebski, (Tue Jun 10, 8:09 am)
Re: [PATCH 0/4] remote show/prune improvement, Olivier Marin, (Tue Jun 10, 9:10 am)
Re: [PATCH 0/4] remote show/prune improvement, Jakub Narebski, (Tue Jun 10, 10:11 am)