Hi, Alan,

On Jan 10, 2011, at 4:50 PM, Alan W. Irwin wrote:

> I confirm those Id difference as well with that svn diff command, e.g.,
> 
> -# $Id:$
> +# $Id$
> 
> That implies that colon was removed. And yet we have
> 
> softw...@raven> grep Id examples/tcl/x24
> # $Id: x24 11447 2011-01-07 07:46:45Z arjenmarkus $
> 
> i.e., the colon is still there.

I think is is an issue of whether the ':' is part of the keyword or part of the 
expansion.  AFAIK, it's supposed to be part of the expansion.  I suspect that 
some svn versions were/are inconsistent with other svn versions wrt this.

> As of revision 11472 I believe I have fixed all these
> issues, but please do whatever you did before to double-check me.

Looks good except for a file added today...

examples/c++/x33.cc

I'm not sure we want keyword expansion for these files...

cmake/external/libqhull/src/libqhull.h
cmake/external/libqhull/src/mem.h
cmake/external/libqhull/src/unix.c

> I note when I changed properties I also got
> 
> -# $Id:$
> +# $Id$
> 
> differences as a signal for that property change.  But that doesn't
> explain the 5.9.7 release differences above (since those should
> involve no property changes)

I don't think that is a signal for that property change since it almost 
certainly won't add the colon back if you remove Id from svn:keywords.  I think 
svn is using the commit as a chance to "correct" the erroneous inclusion of the 
':'.  I think doing "svn cp" to create a tag, like the v5_9_7 tag, is another 
time when svn will "correct" these.

Dave

P.S.  Thanks for fixing my typos.  I'll spell check next time.  :-)


------------------------------------------------------------------------------
Gaining the trust of online customers is vital for the success of any company
that requires sensitive data to be transmitted over the Web.   Learn how to 
best implement a security strategy that keeps consumers' information secure 
and instills the confidence they need to proceed with transactions.
http://p.sf.net/sfu/oracle-sfdevnl 
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to