[PATCH stable] keyword: the CVS keyword is $RCSfile$, not $RCSFile$

Matt Mackall mpm at selenic.com
Fri Nov 27 17:08:59 CST 2009


On Thu, 2009-11-26 at 12:41 +0100, Christian Ebert wrote:
> * Christian Ebert on Thursday, November 26, 2009 at 11:34:25 +0100
> > # HG changeset patch
> > # User Christian Ebert <blacktrash at gmx.net>
> > # Date 1259229077 -3600
> > # Branch stable
> > # Node ID 9ac7d9f8d6eb6c8f400bd5802d38ad2748648840
> > # Parent  b22ff29a3fbb1c1841c1c5b62f510c5ca2c782c3
> > keyword: the CVS keyword is $RCSfile$, not $RCSFile$
> > 
> > http://cvsbook.red-bean.com/cvsbook.html#List%20Of%20Keywords
> > 
> > Fix default keyword map accordingly.
> > Keep $RCSFile$ for Mercurial backwards compatibility.
> 
> Do you think $RCSFile$ could be deprecated/omitted after a
> transition period? There's always the possibility to specify the
> old default [keywordmaps] manually.

It's not really worth the trouble. And the trouble is that 10 years from
now, someone is going to try to check out and build an old project
revision with the old keyword and it's going to fail to build, and the
original engineering team that would have been able to figure out why
will be long gone. 

The maintenance overhead of this one extra line of code is negligible.
It's much lower than the cost of, say, writing this email. But it will
only take one instance of a regression to make the cost of removing it
fairly substantial.

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




More information about the Mercurial-devel mailing list