[PATCH] graphlog: add special char for public and secret phases

Idan Kamara idankk86 at gmail.com
Wed Jan 18 10:48:58 CST 2012


On Wed, Jan 18, 2012 at 6:38 PM, Alain Leufroy <alain.leufroy at logilab.fr>wrote:

> On Wed, 18 Jan 2012, Idan Kamara wrote:
>
>
> > On Wed, Jan 18, 2012 at 5:57 PM,  <alain.leufroy at logilab.fr> wrote:
> > > # HG changeset patch
> > > # User Alain Leufroy <alain.leufroy at logilab.fr>
> > > # Date 1326896559 -3600
> > > # Node ID 43bd418c215edf2a230b3420972a877509157e07
> > > # Parent  2642ac9d28d1375eb8ca92baafa05c02d59a84c6
> > > graphlog: add special char for public and secret phases
> >
> > Users who don't care about phases will find this annoying.
> >
> > It should be at least toggle-able and turned off by default.
> >
>
> Yep ... but notice that there are only minor effective modifications on the
> graph for user who don't care about phases. Look at the tests to be
> convinced of
> this. Allmost modified tests plays with phases.


Changing 'o' to 'O' is actually quite noticeable for something that's trying
to be as compact as possible.

It would actually make more sense for 'o' to correspond to the public phase,
which should account for the majority of revisions in a repo (if I followed
things correctly).

Even so, I still think it's a good idea for this to be toggle-able.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20120118/351bac61/attachment.html>


More information about the Mercurial-devel mailing list