Re: [PATCH] Lose perl dependency. (fwd)

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Jan 20, 2007, 01:18 UTC
Message-ID: <Pine.LNX.4.63.0701200213020.22628@wbgn013.biozentrum.uni-wuerzburg.de>
In-Reply-To: <7vvej2bkn2.fsf@assigned-by-dhcp.cox.net>

Hi,

On Fri, 19 Jan 2007, Junio C Hamano wrote:

> In other words, if it does not make sense for format-patch to
> take --reverse, maybe we should keep it as an internal option,
> just like git-show is the only user of no-walk.

Or we just check in cmd_format_patch() (and possibly other users where --reverse does not make sense) for revs->reverse and die() upon it.

OTOH we can expect people _not_ to use --reverse with format-patch when they don't know what it does! I mean, I don't go and use "ls" with an option I saw in the man page, just because it has a cool ring to it.

Ciao, Dscho

P.S.: Perhaps you should just stop worrying and learn to love --reverse ;-)