Re: svn commit: rev 6453 - in trunk/tools/cvs2svn: . test-data/bogus-tag-cvsrepos test-data/bogus-tag-cvsrepos/CVSROOT

On Fri, Jul 11, 2003 at 05:20:08PM -0500, kfogel@tigris.org wrote:
> + # CVS tag names must start with a letter, and contain only
> + # letters, digits, hyphens, and underscores. It turned out to
> + # be surprisingly hard to find a reference for this on the
> + # Internet; I finally managed to track one down at:
> + #
> + # http://cvsbook.red-bean.com/cvsbook.html#Marking_A_Moment_In_Time__Tags_
> + #
> + # I'm not sure I trust that guy, though, so if anyone has better
> + # reference, please put it here. -kfogel
> + #

Well, he literally wrote the book on CVS so maybe you should cut him
just a little bit a slack. :-)

Some quick testing shows me that my CVS (1.10 'Halibut') does indeed
allow '/' in tags.

Here are some interesting messages, though:

$ cvs tag . foo
cvs [tag aborted]: tag `.' must start with a letter

$ cvs tag a. foo
cvs [tag aborted]: tag `a.' must not contain the characters `$,.:;@'

This is unfortunate, since unix filesystems do not allow '/' in filenames,
and we like to lay things out as .../tags/tagname/.

I see a few options:

1.) catch the '/' and throw a fatal error, recommending that the user
    does "cvs rtag -r old/bad/tag/name new_spiffy_tag_name module" and
    tries again.

2.) catch the '/' and throw a fatal error, unless a "tag name mapping"
    file/option is present that lets cvs2svn know what to use instead

3.) map the '/' to another character. I propose '.', in particular,
    because it is not allowed in CVS tag names, so this mapping
    will not cause any collisions.

--ben

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Jul 12 16:52:41 2003