[PATCH 3 of 4] templates: add 'bisect' keyword to return a cset's bisect status

Yann E. MORIN yann.morin.1998 at anciens.enib.fr
Thu Sep 22 15:13:30 CDT 2011


On Thursday 22 September 2011 21:08:07 Matt Mackall wrote:
> On Thu, 2011-09-22 at 02:10 +0200, Yann E. MORIN wrote:
> > # HG changeset patch
> > # User "Yann E. MORIN" <yann.morin.1998 at anciens.enib.fr>
> > # Date 1316648161 -7200
> > # Node ID b06397d18c1bef158b180cd30bc887d6f77ee62f
> > # Parent  38490dd8eadf5c5e8b77ad2582328d6a4caf1f86
> > templates: add 'bisect' keyword to return a cset's bisect status
> > 
> > This new 'bisect' keyword expands to a cset's bisection status (good, bad
> > and so on...). There is also a new 'shortbisect' filter that yields a
> > single char representing the cset's bisection status.
> > 
> > It uses the two recently-added hbisect.label() and .shortlabel() functions.
> > 
> > Example output using the repository in test-bisect2.t, and some made-up
> > state of the 'end at merge' test (with graphlog, it's so explicit):
> > 
> >   $ hg glog --template '{rev}:{node|short} {bisect}\n'  \
> >             -r 'bisect(range)|bisect(ignored)'
> >   o  17:228c06deef46: bad
> >   |
> >   o  16:609d82a7ebae: bad (implicit)
> 
> What's the '(implicit)' here about?

'bad':
  explicitly marked as 'bad' by the user

'bad (implicit)':
  bad because it belongs to the (bad::bad) set, but not explictly marked
  as 'bad' by the user

So, the user knows exactly what caused the bisection status: his marking
some cset as bad (or good), and bisect excluding some cset because they
are implictly bad (or good).

Having the cset simply marked as 'bad' in either case would be confusing
to the user. It would be confusing to me, at least.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'


More information about the Mercurial-devel mailing list