On Wed, Sep 3, 2014 at 9:32 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Johan Herland <johan@xxxxxxxxxxx> writes: >> diff --git a/builtin/revert.c b/builtin/revert.c >> index f9ed5bd..831c2cd 100644 >> --- a/builtin/revert.c >> +++ b/builtin/revert.c >> @@ -91,6 +91,7 @@ static void parse_args(int argc, const char **argv, struct replay_opts *opts) >> N_("option for merge strategy"), option_parse_x), >> { OPTION_STRING, 'S', "gpg-sign", &opts->gpg_sign, N_("key-id"), >> N_("GPG sign commit"), PARSE_OPT_OPTARG, NULL, (intptr_t) "" }, >> + OPT_BOOL('n', "no-verify", &opts->no_verify, N_("bypass pre-commit hook")), > > I doubt we want this option to squat on '-n'; besides, it is already > taken by a more often used "--no-commit". Oops. I never meant to squat on '-n'. Will fix in re-roll. ...Johan -- Johan Herland, <johan@xxxxxxxxxxx> www.herland.net -- 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