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 ;-) - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html