Ramkumar Ramachandra <artagnon@xxxxxxxxx> writes: > Hi Junio, > > Junio C Hamano writes: >> Ramkumar Ramachandra <artagnon@xxxxxxxxx> writes: >> >> > Since the '-r' command-line option is a no-op provided only for >> > backward compatiblity, don't advertise it in the default usage. >> > >> > Signed-off-by: Ramkumar Ramachandra <artagnon@xxxxxxxxx> >> > --- >> >> Thanks. >> >> It would have saved me a lot of time if you wrote that this option has >> been a no-op since abd6970 (cherry-pick: make -r the default, 2006-10-05), >> but the commit didn't remove it from the usage string, and it somehow >> survived 9509af6 (Make git-revert & git-cherry-pick a builtin, 2007-03-01) >> and later f810379 (Make builtin-revert.c use parse_options, 2007-10-07), >> and it is about time to stop advertising it. > > Thanks. I didn't realize there was so much history to write about. Not necessarily in the final commit message; before "---" would have been sufficient. > Has you queued this somewhere? Sorry, after spending time to dig the history to see when we made this a no-op to ensure that it was sufficiently long time ago and wrote my finding in the message to you, I forgot to apply the patch. Will queue. Thanks. -- 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