Should CVS keyword substitution be permitted?

[Jesse Glick]

Not sure if anyone else cares much about this, but is there a reason 
other Ant committers permit sources in CVS to use keywords such as 
"$Revision$"? Just reminded by the recent <xslt> patch and the bogus

@@ -36,7 +36,7 @@
  * Processes a set of XML documents via XSLT. This is
  * useful for building views of XML based documentation.
  *
- * @version $Revision: 1.90 $
+ * @version $Revision: 1.78.2.7 $
  *
  * @since Ant 1.1
  *
which will have to be pulled out in order to apply the patch.
The @version here (XSLTProcess.java) doesn't seem to serve much purpose 
for either Javadoc or source browsing - has no apparent relation to the 
Ant release, and if you can look up the file's history in CVS you 
probably didn't need to see it here anyway. All it does is produce noise 
when diffing and potential conflicts when merging.

Affected files:
$ find * -type f \! -name '.#*' -exec egrep -q 
'\$(Author|Date|Header|Id|Name|Locker|Log|RCSfile|Revision|Source|State)[:$]' 
{} \; -print
docs/ant2/features.html
docs/manual/credits.html
docs/manual/Integration/jext-plugin.html
docs/manual/OptionalTasks/ejb.html
docs/manual/OptionalTasks/vss.html
proposal/sandbox/svn/src/main/org/apache/tools/ant/taskdefs/svn/SvnChangeLogParser.java
proposal/sandbox/svn/src/testcases/org/apache/tools/ant/taskdefs/svn/AbstractSvnTaskTest.java
proposal/xdocs/src/org/apache/ant/xdoclet/TaskDefPropertiesSubTask.java
proposal/xdocs/src/org/apache/ant/xdoclet/TaskDescriptorSubTask.java
src/etc/testcases/taskdefs/optional/antlr/java.g
src/etc/testcases/taskdefs/optional/antlr/java.tree.g
src/main/org/apache/tools/ant/Project.java
src/main/org/apache/tools/ant/input/DefaultInputHandler.java
src/main/org/apache/tools/ant/input/InputHandler.java
src/main/org/apache/tools/ant/input/InputRequest.java
src/main/org/apache/tools/ant/input/MultipleChoiceInputRequest.java
src/main/org/apache/tools/ant/input/PropertyFileInputHandler.java
src/main/org/apache/tools/ant/taskdefs/AntStructure.java
src/main/org/apache/tools/ant/taskdefs/Basename.java
src/main/org/apache/tools/ant/taskdefs/BuildNumber.java
src/main/org/apache/tools/ant/taskdefs/CVSPass.java
src/main/org/apache/tools/ant/taskdefs/ConditionTask.java
src/main/org/apache/tools/ant/taskdefs/Copy.java
src/main/org/apache/tools/ant/taskdefs/DependSet.java
src/main/org/apache/tools/ant/taskdefs/Dirname.java
src/main/org/apache/tools/ant/taskdefs/Javac.java
src/main/org/apache/tools/ant/taskdefs/Move.java
src/main/org/apache/tools/ant/taskdefs/FixCRLF.java
src/main/org/apache/tools/ant/taskdefs/Sync.java
src/main/org/apache/tools/ant/taskdefs/XSLTProcess.java
src/main/org/apache/tools/ant/taskdefs/condition/And.java
src/main/org/apache/tools/ant/taskdefs/condition/Condition.java
src/main/org/apache/tools/ant/taskdefs/condition/ConditionBase.java
src/main/org/apache/tools/ant/taskdefs/condition/Contains.java
src/main/org/apache/tools/ant/taskdefs/condition/Equals.java
src/main/org/apache/tools/ant/taskdefs/condition/FilesMatch.java
src/main/org/apache/tools/ant/taskdefs/condition/IsReference.java
src/main/org/apache/tools/ant/taskdefs/condition/IsSet.java
src/main/org/apache/tools/ant/taskdefs/condition/Not.java
src/main/org/apache/tools/ant/taskdefs/condition/Or.java
src/main/org/apache/tools/ant/taskdefs/condition/Os.java
src/main/org/apache/tools/ant/taskdefs/cvslib/CVSEntry.java
src/main/org/apache/tools/ant/taskdefs/cvslib/ChangeLogParser.java
src/main/org/apache/tools/ant/taskdefs/cvslib/ChangeLogTask.java
src/main/org/apache/tools/ant/taskdefs/cvslib/ChangeLogWriter.java
src/main/org/apache/tools/ant/taskdefs/cvslib/CvsUser.java
src/main/org/apache/tools/ant/taskdefs/cvslib/RCSFile.java
src/main/org/apache/tools/ant/taskdefs/cvslib/RedirectingStreamHandler.java
src/main/org/apache/tools/ant/taskdefs/cvslib/CvsTagDiff.java
src/main/org/apache/tools/ant/taskdefs/optional/extension/resolvers/AntResolver.java
src/main/org/apache/tools/ant/taskdefs/optional/extension/resolvers/LocationResolver.java
src/main/org/apache/tools/ant/taskdefs/optional/extension/resolvers/URLResolver.java
src/main/org/apache/tools/ant/taskdefs/optional/extension/Compatability.java
src/main/org/apache/tools/ant/taskdefs/optional/extension/Compatibility.java
src/main/org/apache/tools/ant/taskdefs/optional/extension/DeweyDecimal.java
src/main/org/apache/tools/ant/taskdefs/optional/extension/Extension.java
src/main/org/apache/tools/ant/taskdefs/optional/extension/ExtensionAdapter.java
src/main/org/apache/tools/ant/taskdefs/optional/extension/ExtensionResolver.java
src/main/org/apache/tools/ant/taskdefs/optional/extension/ExtensionSet.java
src/main/org/apache/tools/ant/taskdefs/optional/extension/ExtensionUtil.java
src/main/org/apache/tools/ant/taskdefs/optional/extension/ExtraAttribute.java
src/main/org/apache/tools/ant/taskdefs/optional/extension/LibFileSet.java
src/main/org/apache/tools/ant/taskdefs/optional/extension/LibraryDisplayer.java
src/main/org/apache/tools/ant/taskdefs/optional/extension/Specification.java
src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitVersionHelper.java
src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java
src/main/org/apache/tools/ant/taskdefs/optional/net/TelnetTask.java
src/main/org/apache/tools/ant/taskdefs/optional/sound/AntSoundPlayer.java
src/main/org/apache/tools/ant/taskdefs/optional/sound/SoundTask.java
src/main/org/apache/tools/ant/taskdefs/optional/ssh/SSHExec.java
src/main/org/apache/tools/ant/taskdefs/optional/unix/Symlink.java
src/main/org/apache/tools/ant/taskdefs/Execute.java
src/main/org/apache/tools/ant/types/DTDLocation.java
src/main/org/apache/tools/ant/types/Description.java
src/main/org/apache/tools/ant/types/ResourceLocation.java
src/main/org/apache/tools/ant/types/resolver/ApacheCatalog.java
src/main/org/apache/tools/ant/types/resolver/ApacheCatalogResolver.java
src/main/org/apache/tools/ant/util/DateUtils.java
src/main/org/apache/tools/ant/util/TaskLogger.java
src/main/org/apache/tools/ant/util/facade/FacadeTaskHelper.java
src/main/org/apache/tools/ant/util/facade/ImplementationSpecificArgument.java
src/main/org/apache/tools/ant/util/regexp/RegexpFactory.java
src/main/org/apache/tools/ant/util/FileUtils.java
src/main/org/apache/tools/mail/ErrorInQuitException.java
src/main/org/apache/tools/zip/AsiExtraField.java
src/main/org/apache/tools/zip/ExtraFieldUtils.java
src/main/org/apache/tools/zip/UnixStat.java
src/main/org/apache/tools/zip/UnrecognizedExtraField.java
src/main/org/apache/tools/zip/ZipEntry.java
src/main/org/apache/tools/zip/ZipExtraField.java
src/main/org/apache/tools/zip/ZipFile.java
src/main/org/apache/tools/zip/ZipLong.java
src/main/org/apache/tools/zip/ZipOutputStream.java
src/main/org/apache/tools/zip/ZipShort.java
src/testcases/org/apache/tools/ant/taskdefs/condition/ContainsTest.java
src/testcases/org/apache/tools/ant/taskdefs/condition/EqualsTest.java
src/testcases/org/apache/tools/ant/taskdefs/AbstractCvsTaskTest.java
src/testcases/org/apache/tools/ant/taskdefs/AntTest.java
src/testcases/org/apache/tools/ant/taskdefs/BUnzip2Test.java
src/testcases/org/apache/tools/ant/taskdefs/BZip2Test.java
src/testcases/org/apache/tools/ant/taskdefs/ChecksumTest.java
src/testcases/org/apache/tools/ant/taskdefs/GUnzipTest.java
src/testcases/org/apache/tools/ant/taskdefs/JavacTest.java
src/testcases/org/apache/tools/ant/taskdefs/RmicTest.java
src/testcases/org/apache/tools/ant/taskdefs/SubAntTest.java
src/testcases/org/apache/tools/ant/taskdefs/TypedefTest.java
src/testcases/org/apache/tools/ant/taskdefs/UntarTest.java
src/testcases/org/apache/tools/ant/taskdefs/optional/ReplaceRegExpTest.java
src/testcases/org/apache/tools/ant/taskdefs/optional/junit/JUnitVersionHelperTest.java
src/testcases/org/apache/tools/ant/taskdefs/optional/junit/NoVmCrash.java
src/testcases/org/apache/tools/ant/taskdefs/optional/junit/Printer.java
src/testcases/org/apache/tools/ant/taskdefs/optional/junit/Sleeper.java
src/testcases/org/apache/tools/ant/taskdefs/optional/junit/VmCrash.java
src/testcases/org/apache/tools/ant/taskdefs/optional/unix/SymlinkTest.java
src/testcases/org/apache/tools/ant/taskdefs/RecorderTest.java
src/testcases/org/apache/tools/ant/types/XMLCatalogBuildFileTest.java
STATUS

Background:
https://www.cvshome.org/docs/manual/cvs-1.11.19/cvs_12.html
Comments?
-J.
--
Jesse Glick <mailto:[EMAIL PROTECTED]> x22801
NetBeans, Open APIs  <http://www.netbeans.org/>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]