[patch #4573] Fix for keyword expansion problem/mis-feature during commit

[Todd Denniston]

Follow-up Comment #7, patch #4573 (project cvs):

I believe that the behavior should be left as is for the following reasons:

1) it is a core behavior that was inherited from RCS.

2) people who are inexperienced using tools simply need educated, i.e., they
should have to read appropriate parts of the manual for any tool they use,
and appropriate should be chosen by their task leaders.

3) it is the documented behavior... RCS key words (which is what these are)
exist for the explicit reason of having the Version Control System (VCS) set
them any time the VCS interacts with the file.
from the Keyword substitution section of the manual[1]:
"Embedded strings of the form $keyword$ and $keyword:...$ in a file are
replaced with strings of the form $keyword:value$ whenever you obtain a new
revision of the file."
"CVS will automatically ... expand the string as part of the commit
operation."

4) in version 1.12.x local keywords can be created for your users to mess
with[2].

5) in version 1.12.x keyword expansion can be controlled in a more fine
grained approach[3] already.

6) if the users need something that will not be overwritten by the VCS then
create that, these keywords were not meant for that purpose.

7) the manual gives appropriate methods to control the substitution, these
methods can be put in the users .cvsrc (AFAIK) if the users do not want to
remember to add the substitution method of choice each time they type a
command.[4][5]

[1] http://ximbiot.com/cvs/manual/cvs-1.11.21/cvs_12.html#SEC97
1.12.x manual has the same information for all the keyword information that
exists in 1.11.x but also has some added options (from what I could see).

[2] http://ximbiot.com/cvs/manual/cvs-1.12.13/cvs_12.html#SEC101
[3] http://ximbiot.com/cvs/manual/cvs-1.12.13/cvs_12.html#SEC104
[4] http://ximbiot.com/cvs/manual/cvs-1.11.21/cvs_12.html#SEC101
[5] http://ximbiot.com/cvs/manual/cvs-1.11.21/cvs_16.html#SEC115

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?func=detailitem&item_id=4573>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/