[Kolab-devel] CVS ID tags.

[Bo Thorsen]

On Tuesday 09 November 2004 16:37, David Faure wrote:
> On Tuesday 09 November 2004 14:40, Bernhard Reiter wrote:
> > On Monday 08 November 2004 09:14, Bo Thorsen wrote:
> > > On Friday 05 November 2004 13:39, cvs at intevation.de wrote:
> > > > Update of /kolabrepository/doc/kde-client/cvs-instructions
> > > > In directory doto:/tmp/cvs-serv19469
> > > >
> > > > Modified Files:
> > > >  checkout_kde_i18n_initial.sh checkout_kde_i18n_proko2.sh
> > > >  checkout_proko2.sh
> > > > Log Message:
> > > > Added CVS Id tag in commented out second line.
> > >
> > > What made you do that?
> >
> > I have those files lying around at several places and want directly
> > see what revision they are.
> >
> > >The Id tag does no good at all and hurts branching.
> >
> > Why does it hurt branching, because of the merge back?
>
> Yes. When diffing, when merging, when importing into another CVS
> and doing diffs and merge, etc.
>
> In the KDE CVS we disallowed it:
> http://developer.kde.org/policies/commitpolicy.html#19
>
> But on other hand for such simple scripts, I think they are a good
> idea. Just yesterday I was comparing foo and foo.rpmnew after a
> mandrake-10.1 upgrade, and having the revision numbers in the actual
> scripts helped a lot ensuring which one was the most recent one.

Actually all of it would be without issues if only cvs were sane enough to 
exclude the tag contents from a cvs diff. Then there would be no issues 
at all in having it in there. Unfortunately thats just yet another place 
where cvs falls short of being a reasonable solution :-(

Bo.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.kolab.org/pipermail/devel/attachments/20041110/f97c837a/attachment.sig>