Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH] revert: Hide '-r' option in default usage

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:51:02

Ramkumar Ramachandra [off-list ref] writes:
Hi Junio,

Junio C Hamano writes:
quoted
Ramkumar Ramachandra [off-list ref] writes:
quoted
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 <redacted>
---
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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help