[Kolab-devel] CVS ID tags.

[David Faure]

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.

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).