[PATCH evolve-ext] fold: add argument to handle ambiguous case

[Pierre-Yves David]

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 
anything.

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.


-- 
Pierre-Yves David