From: Johannes Schindelin Date: Sat, 20 Jan 2007 01:18:08 GMT Subject: Re: [PATCH] Lose perl dependency. (fwd) Message-ID: 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 ;-)