Re: [PATCH] git-rebase.sh: fix typos in error messages
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Sep 28, 2018, 22:05 UTC
- Message-ID
- <xmqqsh1t9t27.fsf@gitster-ct.c.googlers.com>
- In-Reply-To
- <xmqqzhw19t9a.fsf@gitster-ct.c.googlers.com>
Junio C Hamano <gitster@pobox.com> writes:
Show 7 quoted lines
> ... However, because the same mistakes are inherited to
> builtin/rebase.c by these topics, we'd need a matching fix to
> correct 07664161 ("builtin rebase: error out on incompatible
> option/mode combinations", 2018-08-08) and either squash the fix
> into that commit, or queue it on top of pk/rebase-in-c-5-test topic.
>
> Will queue; thanks.Here is what I'd queue, too.
-- >8 -- Subject: [PATCH] rebase: fix typos in error messages
The separator between words in a multi-word option name is a dash, not an underscore.
Inspired by a matching change by Ralf Thielow for the scripted version of "git rebase".
Signed-off-by: Junio C Hamano <gitster@pobox.com> --- builtin/rebase.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/builtin/rebase.c b/builtin/rebase.c index 1a697d70c9..0f9a40aae5 100644 --- a/builtin/rebase.c +++ b/builtin/rebase.c @@ -1135,15 +1135,15 @@ int cmd_rebase(int argc, const char **argv, const char *prefix) * git-rebase.txt caveats with "unless you know what you are doing" */ if (options.rebase_merges) - die(_("error: cannot combine '--preserve_merges' with " + die(_("error: cannot combine '--preserve-merges' with " "'--rebase-merges'")); if (options.rebase_merges) { if (strategy_options.nr) - die(_("error: cannot combine '--rebase_merges' with " + die(_("error: cannot combine '--rebase-merges' with " "'--strategy-option'")); if (options.strategy) - die(_("error: cannot combine '--rebase_merges' with " + die(_("error: cannot combine '--rebase-merges' with " "'--strategy'")); }
-- 2.19.0-271-gfe8321ec05