[PATCH v2 7/7] bisect: allows any terms set by user

From: Antoine Delaite <antoine.delaite@ensimag.grenoble-inp.fr>
Date: Jun 16, 2015, 21:07 UTC
Message-ID: <1482001285.562840.1434488820667.JavaMail.zimbra@ensimag.grenoble-inp.fr>
In-Reply-To: <vpq8ubl73py.fsf@anie.imag.fr>

Matthieu Moy <Matthieu.Moy@grenoble-inp.fr> writes:

Show 10 quoted lines

>I would say "for the current bisection", i.e. 
> 
>$ git bisect start 
>$ git bisect terms foo bar # Scope starts here 
>... 
>The first 'bar' commit is deadbeef. # Scope ends here 
> 
>$ git bisect terms foo bar 
>You need to start by "git bisect start" 
>Do you want me to do it for you [Y/n]?

I personnaly don't like this autostart. In the first version of this patch 7/7 'git bisect terms' has to be called before the start. I think it is better because we can then use 'git bisect start rev1 rev2 ...'

Show 7 quoted lines

>> The next 'bisect start rev1 rev2' would be in bad/good mode. But this 
>> have to be discuted, do the user have to type 'git bisect terms' each 
>> bisection if he wants to use special terms ? 
> 
>To me, yes. If we allow arbitrary terms, then they will most likely be 
>specific to one bisect session (e.g. if I bisect "present/absent" once, 
>it tells me nothing about what I'll want for my next bisection).

Ok.

Thanks.