[PATCH 3 of 4] log: add a marker for closed heads

Matt Mackall mpm at selenic.com
Wed May 12 10:56:23 CDT 2010


On Wed, 2010-05-12 at 10:15 +0200, Gilles Moris wrote:
> # HG changeset patch
> # User Gilles Moris <gilles.moris at free.fr>
> # Date 1273478166 -7200
> # Node ID bbef7225b4770ac633a804feb990e5b3c88e7bed
> # Parent  fc60860d838f205830014de60bf9eba1bb65735d
> log: add a marker for closed heads
> 
> This will appear as a optional separate line below the "changeset:" line and
> before the "branch:" line. There is a leading space, so that the template
> keyword {closed} can be appended to an existing line, or appear as kind of
> line continuation.

Again, my plea to show us what it looks like up front.

> changeset:   11:c84627f3c15d
> + (closed)
>  branch:      b
>  user:        test
>  date:        Thu Jan 01 00:00:09 1970 +0000

Yuck!

Folks, you have to be more imaginative. And by that, I mean you're
failing to imagine just how stupid and LAZY your fellow programmers and
admins have already been when coding up hacks IN SHELL SCRIPTS to parse
these things for their build systems after we've guaranteed they won't
change.

We can't stick a new line in there. And we definitely can't stick in a
line without a colon! Nor can it go at the end of any existing
fixed-form lines. I frankly don't think we can fix this without a new
log option or sticking it under --debug.

For now, please send me just the code that implements a new template
element (and not the modified templates).

-- 
Mathematics is the supreme nostalgia of our time.




More information about the Mercurial-devel mailing list