[PATCH evolve-ext] fold: add argument to handle ambiguous case
pierre-yves.david at ens-lyon.org
Mon Jun 8 19:08:05 CDT 2015
On 06/02/2015 08:35 PM, Durham Goode wrote:
> On 6/2/15 8:32 PM, Durham Goode wrote:
>> If a user specifies a single commit (via revset, or node, or
>> whatever), can we error out and say "you only specified one commit.
>> did you mean: hg fold X::." ? That keeps the syntax precise (fold
>> exactly what I say, don't compute any closures), and teaches the user
>> about revsets.
> Heck, if X is the immediate parent, you could even suggest "hg fold 'X +
> .'" so it's extremely obvious what the correct syntax is (no
> understanding of :: necessary)
I like the idea of customizing the error message instead of the
behavior. It allow to provide useful information while not blowing up
I would be pretty unhappy to expose random user to X::Y recommendation
because they will screw it up a X:Y and get burned down. So the idea of
X+. is also interesting (but won't work for a full stack).
So I like the ideas above and that should definitely goes to the wiki
page that sums up the discussion. However, this does not address the
issue of "the common case is more complicated that it should be, so I'm
not 100% sold.
More information about the Mercurial-devel