named heads

Matt Mackall mpm at selenic.com
Sun Jul 26 13:12:20 CDT 2009


On Sun, 2009-07-26 at 09:51 -0400, Faheem Mitha wrote:
> Dear People,
> 
> I've been reviewing and testing hgs named heads, with a view to using them. I 
> have a few comments.

There's no such thing as 'named heads'. 

> 1) hg view branchname
> 
> works as expected (shows only changesets corresponding to branchname), but is 
> undocumented. Also, I cannot find any discussion of this option.

hg help revisions

There's no special branch magic in hg view, which is more or less
unmaintained by this point anyway.

> 2) I have a repos with two heads, one of them corresponding to a branch (temp) 
> which is closed. I get
> 
> faheem at orwell:/tmp/test$ hg heads -a
> 
> changeset:   8:db4fd6f88ebb
> branch:      temp
> tag:         tip
> [...]
> summary:     Close branch temp.
> 
> changeset:   3:98c0d9bde0f2
> parent:      1:f7cf0b6e18f1
> [...]
> summary:     Add foobar.
> 
> The -a option clearly states
> 
> -a --active    show only the active branch heads from open branches
> 
> so this is wrong.

Yes, the description is wrong. The body of the help however, explicitly
defines 'active branch heads' in a way that is synonymous with heads. So
heads -a without a branch name is redundant.

> 4) In hg help branches below
> 
> a) Why is only -a in brackets and not -c?

Send a patch.

> b) It says
> 
> "List the repository's named branches, indicating which ones are inactive."
> 
> This is immediately followed by
> 
> "If -c/--closed is specified, also list branches which have been marked 
> closed."
> 
> So, in fact, the help should read something like
> 
> "List the repository's open named branches, indicating which ones are 
> inactive."

Send a patch.

This email is way too long and raises way too many issues that have
already been discussed here in depth that I don't have the time or
patience to address the rest of it.

-- 
http://selenic.com : development and support for Mercurial and Linux




More information about the Mercurial mailing list