[PATCH] revert: simplify help message, better error message on reverting a merge

Kevin Bullock kbullock+mercurial at ringworld.org
Wed Jun 8 10:50:24 CDT 2011


On Jun 8, 2011, at 10:47 AM, Pierre-Yves David wrote:

> On Wed, Jun 08, 2011 at 10:16:19AM -0500, Kevin Bullock wrote:
>> On Jun 8, 2011, at 3:08 AM, Pierre-Yves David wrote:
>> 
>>> On Tue, Jun 07, 2011 at 03:52:21PM -0500, Matt Mackall wrote:
>>> 
>>>> This one is rather notable in that it no longer considers trying to
>>>> revert in a merge to be ambiguous, it just uses the first parent:
>>>> 
>>>> http://www.selenic.com/hg/rev/afe0d4c24866
>>> 
>>> Considering that rebase switch parent it might be useful to specify which
>>> parent was used for the revert.
>> 
>> I'm confused. How does rebase enter into this?
> 
> Rebase do merge. For rebase, local is "what you rebase on" other is "what your
> are currently rebasing". This confuse several people.


But once it's rebased, a changeset has only one parent. How would rebase's merging affect revert?

In any case, I don't think it's sensible to include anything about rebase (an extension) in the help for revert (a core command).

pacem in terris / mir / shanti / salaam / heiwa
Kevin R. Bullock



More information about the Mercurial-devel mailing list