Re: [PATCH] review requested, add a feature to mergemaster

At 2:53 PM +0100 1/12/05, Anton Berezin wrote:
>Following the idea by Sune Stjerneby, suggestions from Andrzej
>Tobola and Phil Regnauld, and a bit of discussion on -current,
>I would like to request a review of a new mergemaster feature
>aimed to simplify manual work in some cases.
>
>A new option is added:
>
>  -R cvsroot  For files that contain a $FreeBSD$ string,

Hmm.  I'm not sure how useful this will be (not that I object
to it in anyway, I'm just not sure...).

But let me note:

>===================================================================
>RCS file: /home/ncvs/src/usr.sbin/mergemaster/mergemaster.8,v
>retrieving revision 1.29
>diff -u -r1.29 mergemaster.8

>+.It Fl R Ar cvsroot
>+For files that contain a $FreeBSD$ string,
>+but which does not match the one in the corresponding file
>+in the temporary root,
>+attempt to extract the same version from the
>+.Fx
>+CVS repository specified by
>+.Ar cvsroot.

Be careful when trying to add the literal string of $FreeBSD$
to any documentation file.  This will look fine when you are
testing it, but when you *commit* it, CVS will expand that
string into the FreeBSD-version information for that file.  I
forget what you have to do to get the right results, but you
don't want to have the literal '$FreeBSD$' string in the file.

-- 
Garance Alistair Drosehn            =   [email protected]
Senior Systems Programmer           or  [email protected]
Rensselaer Polytechnic Institute    or  [email protected]
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-audit
To unsubscribe, send any mail to "[email protected]"