Date: Mon, 19 Aug 2002 00:06:18 +0100 From: Josef Karthauser <joe@tao.org.uk> To: Tony Finch <dot@dotat.at> Cc: cvs@freebsd.org, doc@freebsd.org, dwmalone@freebsd.org Subject: Re: CVSROOT documentation patch Message-ID: <20020818230618.GC603@genius.tao.org.uk> In-Reply-To: <20020719204817.C8238@chiark.greenend.org.uk> References: <E17VCWe-000L8c-00@cyan.csi.cam.ac.uk> <20020718163457.D32457@chiark.greenend.org.uk> <20020719202612.B8238@chiark.greenend.org.uk> <20020719204817.C8238@chiark.greenend.org.uk>
next in thread | previous in thread | raw e-mail | index | archive | help
--S1BNGpv0yoYahz37 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Jul 19, 2002 at 08:48:17PM +0100, Tony Finch wrote: > This is an update to the CVSROOT documentation. It removes an unexpanded > $Date$ keyword, adds more crossreferences between the various CVSROOT > files, adds notes about unused functionality that's broken with remote > commits, and explains some compatibility issues related to local FreeBSD > CVS hacks in more detail. >=20 > Should I commit it myself or leave it to a doc committer? >=20 Please commit this yourself, and feel free to commit to this file as much as you want. Thanks Joe > Tony. > --=20 > f.a.n.finch <dot@dotat.at> http://dotat.at/ > NORTH UTSIRE SOUTH UTSIRE: NORTH OR NORTHWEST 3 OR 4, OCCASIONALLY 5. FAI= R. > GOOD. >=20 >=20 > Index: article.sgml > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > RCS file: /home/ncvs/doc/en_US.ISO8859-1/articles/cvs-freebsd/article.sgm= l,v > retrieving revision 1.5 > diff -u -r1.5 article.sgml > --- article.sgml 10 Apr 2002 12:54:25 -0000 1.5 > +++ article.sgml 19 Jul 2002 19:36:21 -0000 > @@ -22,8 +22,6 @@ > <holder role=3D"mailto:stijn@win.tue.nl">Stijn Hoop</holder> > </copyright> > =20 > - <pubdate role=3D"rcs">$Date$</pubdate> > - > <releaseinfo>$FreeBSD: doc/en_US.ISO8859-1/articles/cvs-freebsd/arti= cle.sgml,v 1.5 2002/04/10 12:54:25 kuriyama Exp $</releaseinfo> > =20 > <abstract> > @@ -183,8 +181,9 @@ > <listitem> > <para><filename>checkoutlist</filename> - this files lists all > files under control of <application>CVS</application> in this > - directory. You should edit this to remove some FreeBSD speci= fic > - files.</para> > + directory, apart from the standard ones created by > + <literal>cvs init</literal>. You should edit this to > + remove some FreeBSD specific files.</para> > </listitem> > =20 > <listitem> > @@ -196,18 +195,20 @@ > =20 > <listitem> > <para><filename>commitcheck</filename> - this script is invoked > - directly from <application>CVS</application>. It first check= s if > - the committer has access to the specified part of the tree, = and > - then runs <filename>commit_prep.pl</filename> for the various > - pre-commit checks. If those are OK, <application>CVS</applic= ation> > - will allow the commit to proceed. You should not have to tou= ch > - this file.</para> > + directly from <application>CVS</application>. It first checks > + if the committer has access to the specified part of the tree > + using <filename>cvs_acls.pl</filename>, and then runs > + <filename>commit_prep.pl</filename> for the various pre-comm= it > + checks. If those are OK, <application>CVS</application> will > + allow the commit to proceed. You should not have to touch th= is > + file.</para> > </listitem> > =20 > <listitem> > <para><filename>commitinfo</filename> - this file is used by > <application>CVS</application> to determine which script to = run > - before a commit. You should not have to touch this file.</pa= ra> > + before a commit—in this case <filename>commitcheck</fi= lename>. > + You should not have to touch this file.</para> > </listitem> > =20 > <listitem> > @@ -238,13 +239,16 @@ > <application>CVS</application> to enable or disable keyword > expansion, or whether a file should be considered binary. You > can edit this as you wish. More information about this file > - is available in the <application>CVS</application> manual.</= para> > + is available in the <application>CVS</application> manual. > + Note that the <literal>-t</literal> and <literal>-f</literal> > + options don't work correctly with client/server > + <application>CVS</application></para> > </listitem> > =20 > <listitem> > <para><filename>edithook</filename> - this file is not used > - any more, but kept for historic reasons. You should not have= to > - touch this file.</para> > + any more, but kept for historic reasons. You can safely > + remove this file.</para> > </listitem> > =20 > <listitem> > @@ -252,49 +256,56 @@ > uses this file for editor overrides. FreeBSD does not use th= is > functionality, as parsing the log message is done in the > <filename>verifymsg</filename> and <filename>logcheck</filen= ame> > - files. You should not have to touch this file.</para> > + files. This is because the <filename>editinfo</filename> > + functionality doesn't work properly with remote commits, or = ones > + that use the <literal>-m</literal> or <literal>-F</literal> > + options. You should not have to touch this file.</para> > </listitem> > =20 > <listitem> > <para><filename>exclude</filename> - this file lists regular > - expressions that are used to determine files which cannot co= ntain a > - revision header. In the FreeBSD setup, all files under revis= ion > - control need to have a revision header (like > - $FreeBSD$). All filenames that match one of th= e lines > - in this file are exempted from this check. You should add > - expressions to this file as you checkin files that cannot ha= ve a > - revision header. For the purpose of installing the scripts, = it > - may be best to exclude <filename>CVSROOT/</filename> from he= ader > - checks.</para> > + expressions that are used by <filename>commit_prep.pl</filen= ame> > + to determine files which cannot contain a revision header. I= n the > + FreeBSD setup, all files under revision control need to have= a > + revision header (like $FreeBSD$). All filename= s that > + match one of the lines in this file are exempted from this c= heck. > + You should add expressions to this file as you checkin files= that > + cannot have a revision header. For the purpose of installing= the > + scripts, it may be best to exclude <filename>CVSROOT/</filen= ame> > + from header checks.</para> > </listitem> > =20 > <listitem> > - <para><filename>log_accum.pl</filename> - this is a script that > - takes the log message as provided by the > - <filename>logcheck</filename> script, and appends it to a lo= g file > - in the repository for backup purposes. It also handles maili= ng out a > - message to an email address you provide (in > - <filename>cfg_local.pm</filename>). You should not have to t= ouch > - this file.</para> > - </listitem> > + <para><filename>log_accum.pl</filename> - this is a script tha= t takes > + the log message as provided by the <filename>logcheck</filen= ame> > + script, and appends it to a log file in the repository for b= ackup > + purposes. It also handles mailing out a message to an email = address > + you provide (in <filename>cfg_local.pm</filename>). It hooks= into > + <application>CVS</application> via <filename>loginfo</filena= me>. > + You should not have to touch this file.</para> > + </listitem> > =20 > <listitem> > <para><filename>logcheck</filename> - this file parses the com= mit > log message that committers provide, and attempts to sanitiz= e it > - somewhat.</para> > + somewhat. It hooks into <application>CVS</application> via > + <filename>verifymsg</filename>. You should not have to touch > + this file.</para> > =20 > <note><para>This script depends on a local FreeBSD hack of > <application>CVS</application>: this version reads the log m= essage > back in after this script has modified it. The stock version= of > - <application>CVS</application> apparently does not, which wo= uld > - make this script useless.</para></note> > + <application>CVS</application> does not do this which makes > + <filename>logcheck</filename> unable to clean up the log mes= sage, > + although it is still able to check that it is syntactically > + OK.</para></note> > </listitem> > =20 > <listitem> > <para><filename>loginfo</filename> - this file is used by > <application>CVS</application> to control where log > - information is sent. You should not have to touch this > - file.</para> > + information is sent; <filename>log_accum.pl</filename> hooks > + in here. You should not have to touch this file.</para> > </listitem> > =20 > <listitem> > @@ -315,7 +326,8 @@ > =20 > <listitem> > <para><filename>options</filename> - this file is specific to > - the FreeBSD version of <application>CVS</application>. It co= ntains > + the FreeBSD version of <application>CVS</application>, and is > + also supported by the Debian version. It contains > the keyword to expand in revision headers. You should alter = this to > match the keyword you specified in > <filename>cfg_local.pm</filename> (if you use that feature, = which > @@ -323,10 +335,11 @@ > </listitem> > =20 > <listitem> > - <para><filename>rcsinfo</filename> - this file maps directorie= s in > - the repository to template files. By default, FreeBSD uses o= ne > - template for the whole repository. You can add others to thi= s file > - if you wish.</para> > + <para><filename>rcsinfo</filename> - this file maps directories > + in the repository to template files such as > + <filename>rcstemplate</filename>. By default, FreeBSD uses o= ne > + template for the whole repository. You can add others to this > + file if you wish.</para> > </listitem> > =20 > <listitem> > @@ -345,30 +358,34 @@ > =20 > <listitem> > <para><filename>taginfo</filename> - this file maps tag operat= ions > - on repository directories to access control scripts. You sho= uld not > - have to touch this file.</para> > + on repository directories to access control scripts such as > + <filename>tagcheck</filename>. You should not have to touch = this > + file.</para> > </listitem> > =20 > <listitem> > - <para><filename>unwrap</filename> - this script is needed to > + <para><filename>unwrap</filename> - this script can be used to > automatically <quote>unwrap</quote> binary files (see > <filename>cvswrappers</filename>) on checkout. It is not use= d in the > - current FreeBSD setup. You should not have to touch this > - file.</para> > + current FreeBSD setup because the functionality it hooks into > + doesn't work well with remote commits. You should not have to > + touch this file.</para> > </listitem> > =20 > <listitem> > <para><filename>verifymsg</filename> - this file maps reposito= ry > - directories to post processor scripts of log messages. You s= hould > - not have to touch this file.</para> > + directories to post processor scripts of log messages such as > + <filename>logcheck</filename>. You should not have to touch > + this file.</para> > </listitem> > =20 > <listitem> > - <para><filename>wrap</filename> - this script is needed to > + <para><filename>wrap</filename> - this script can be used to > automatically <quote>wrap</quote> binary files (see > <filename>cvswrappers</filename>) on checkin. It is not used= in the > - current FreeBSD setup. You should not have to touch this > - file.</para> > + current FreeBSD setup because the functionality it hooks into > + doesn't work well with remote commits. You should not have to > + touch this file.</para> > </listitem> > </itemizedlist> > </sect2> --=20 "As far as the laws of mathematics refer to reality, they are not certain; and as far as they are certain, they do not refer to reality." - Albert Einstein, 1921 --S1BNGpv0yoYahz37 Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.0.7 (FreeBSD) iEYEARECAAYFAj1gKGkACgkQXVIcjOaxUBb8UgCfcjtyMv0ZRw80vecIzU57L+Pm /vcAoN7WFbXuPC1NXlZAt+SzK8p7s74a =UtDo -----END PGP SIGNATURE----- --S1BNGpv0yoYahz37-- To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-doc" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20020818230618.GC603>