threads / bug / 45484

Bug: Revert with -m 0 says no -m was given

Subject: Bug: Revert with -m 0 says no -m was given

## tl;dr

4 messages between Mar 23, 2017 and Mar 23, 2017.

replies: 3people: 3as markdown or json

Ulrik Sverdrup· Mar 23, 2017, 17:17 UTC · lore
Hi,

Git produces a misleading error when you tell it `git revert -m 0`. The user error is that the parent should be greater than zero, but git doesn't help you. (I forgot that the first parent is 1 and not 0.)

$ git revert -m 0 abc error: commit abc... is a merge but no -m option was given. fatal: revert failed

Git already handles too large m correctly:

$ git revert -m 3 abc error: commit abc... does not have parent 3 fatal: revert failed

-Ulrik
Junio C Hamano· Mar 23, 2017, 18:15 UTC · re: Ulrik Sverdrup · lore

Re: Bug: Revert with -m 0 says no -m was given

Ulrik Sverdrup <ulrik.sverdrup@gmail.com> writes:
> Git produces a misleading error when you tell it `git revert -m 0`.
Thanks.  

I think this is the same issue as the one with 'cherry-pick' that was reported earlier. We have a fix in the 'master' branch already; please try it out.

Ævar Arnfjörð Bjarmason· Mar 23, 2017, 20:39 UTC · re: Junio C Hamano · lore

Re: Bug: Revert with -m 0 says no -m was given

On Thu, Mar 23, 2017 at 7:15 PM, Junio C Hamano <gitster@pobox.com> wrote:
Show 9 quoted lines
> Ulrik Sverdrup <ulrik.sverdrup@gmail.com> writes:
>
>> Git produces a misleading error when you tell it `git revert -m 0`.
>
> Thanks.
>
> I think this is the same issue as the one with 'cherry-pick' that
> was reported earlier.  We have a fix in the 'master' branch already;
> please try it out.
Specifically, in master:

$ ./git revert afd6726309 error: commit afd6726309f57f532b4b989a75c1392359c611cc is a merge but no -m option was given. fatal: revert failed $ ./git revert -m 0 afd6726309 error: option `mainline' expects a number greater than zero

Junio C Hamano· Mar 23, 2017, 21:38 UTC · re: Ævar Arnfjörð Bjarmason · lore

Re: Bug: Revert with -m 0 says no -m was given

Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
Show 19 quoted lines
> On Thu, Mar 23, 2017 at 7:15 PM, Junio C Hamano <gitster@pobox.com> wrote:
>> Ulrik Sverdrup <ulrik.sverdrup@gmail.com> writes:
>>
>>> Git produces a misleading error when you tell it `git revert -m 0`.
>>
>> Thanks.
>>
>> I think this is the same issue as the one with 'cherry-pick' that
>> was reported earlier.  We have a fix in the 'master' branch already;
>> please try it out.
>
> Specifically, in master:
>
> $ ./git revert afd6726309
> error: commit afd6726309f57f532b4b989a75c1392359c611cc is a merge but
> no -m option was given.
> fatal: revert failed
> $ ./git revert -m 0 afd6726309
> error: option `mainline' expects a number greater than zero
Yes, exactly.
Thanks.

← back to recent threads