Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CVS tags in source code #41

Closed
GoogleCodeExporter opened this issue Mar 16, 2015 · 9 comments
Closed

CVS tags in source code #41

GoogleCodeExporter opened this issue Mar 16, 2015 · 9 comments

Comments

@GoogleCodeExporter
Copy link

Need to decide what to do with the $Id$ tags in the source code (and other 
tags in the documentation). Apparently it is possible to get Subversion to 
set these automatically, but not sure how. In any case, do we still want 
these?

Original issue reported on code.google.com by lar...@gmail.com on 30 Jun 2009 at 12:00

@GoogleCodeExporter
Copy link
Author

If the Id keyword is still wanted:
http://svnbook.red-bean.com/en/1.4/svn.advanced.props.special.keywords.html

Original comment by lars.he...@gmail.com on 30 Jun 2009 at 12:03

@GoogleCodeExporter
Copy link
Author

This explains how to make it work for a single file, but not how to set the 
property 
for the entire source tree. There is one solution, but seems to require making 
changes to files on the server side plus changes for each individual user: 
http://
fixlinux.com/2006/07/24/subversion-and-keyword-substitution/

Original comment by lar...@gmail.com on 8 Jul 2009 at 4:13

@GoogleCodeExporter
Copy link
Author

You can set SVN properties on a full directory tree, BUT: it only applies it for
*existing* files, not for future added files. 

You need to set the SVN property 'svn:keywords' to a set of:
  URL, HeadURL             - The URL for the head version of the object.
  Author, LastChangedBy    - The last person to modify the file.
  Date, LastChangedDate    - The date/time the object was last modified.
  Rev, Revision,           - The last revision the object changed.
  LastChangedRevision
  Id                       - A compressed summary of the previous 4 keywords.

By using 'svn propset svn:keywords "options" files'
In code you can then do:

/*
  Some file info
  Last change on $Date$ in $Rev$ by $LastChangedBy$
  -- $HeadURL$ --
*/

On 'svn propset' there is an recursive (-R) option for setting it on a 
directory tree.


Original comment by qsieb...@gmail.com on 15 Jul 2009 at 11:09

@GoogleCodeExporter
Copy link
Author

We do need it to be set for future added files, since otherwise people will 
forget 
to set it for those files.

I'm beginning to think that these tags (// $Id$ at the top of each source file) 
are 
more trouble than they are worth (if indeed they are worth anything at all), so 
perhaps we should just remove them from the source and live without them? 
Thoughts 
on that?

Some of these are used in the documentation but we can set the necessary 
properties 
for the doc files that use them and maintain those settings manually, since 
it's 
such a limited number of files.

Original comment by lar...@gmail.com on 16 Jul 2009 at 2:13

@GoogleCodeExporter
Copy link
Author

I'm open to removing $Id$ from subversion. We can always find the information 
in the 
subversion history.

Original comment by indiapaleale@gmail.com on 22 Sep 2009 at 6:19

@GoogleCodeExporter
Copy link
Author

People have had months to speak out if they want to keep them, so I think we 
just 
decide that they have to go.

Original comment by lar...@gmail.com on 22 Sep 2009 at 7:00

@GoogleCodeExporter
Copy link
Author

I've started cleaning up the code in big batches focussing only on the tags. 

Revisions 2126 - 2137 have removed all ID tags from java files

Original comment by qsieb...@gmail.com on 29 May 2011 at 11:35

@GoogleCodeExporter
Copy link
Author

Revision 2139 removed most other tags.

Original comment by qsieb...@gmail.com on 29 May 2011 at 12:57

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author

Original comment by qsieb...@gmail.com on 27 Jan 2012 at 10:36

  • Added labels: Release5.2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant