Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 30 Mar 2004 22:12:10 +0200
From:      "Simon L. Nielsen" <simon@FreeBSD.org>
To:        Mark Linimon <linimon@lonesome.com>
Cc:        eik@FreeBSD.org
Subject:   Re: docs/64144: [patch] document procedure to add a new ports category to Committer's Guide
Message-ID:  <20040330201210.GA749@arthur.nitro.dk>
In-Reply-To: <200403301233.26078.linimon@lonesome.com>
References:  <200403120710.i2C7A146012080@freefall.freebsd.org> <200403301233.26078.linimon@lonesome.com>

next in thread | previous in thread | raw e-mail | index | archive | help

--OgqxwSJOaUobr8KG
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On 2004.03.30 12:33:26 -0600, Mark Linimon wrote:
> Here is a revised patch that incorporates feedback that I got from
> eik@, and my own slightly improved knowledge of both the underlying
> problems, and of DocBook.

Looks good in general, but a few comments :

[CUT]

> +		<step>
> +		  <para>On your own local system, test the proposed
> +		    changes: first, comment out the
> +		    <makevar>SUBDIR</makevar> entries in the old
> +		    ports' categories' <filename>Makefile</filename>s;
> +		    then, enable building the new category in
> +		    <filename>ports/Makefile</filename>.
> +		    Run <command>make checksubdirs</command> in the
> +		    affected category directories to check the
> +		    <makevar>SUBDIR</makevar> entries.  Next, in
> +		    the <filename>ports/</filename> directory, run

Should be :

+		    the <filename class=3D"directory">ports/</filename> directory, run

[CUT]

> +		<step>
> +		  <para>Update the instructions for
> +		    <command>cvsup(1)</command> by modifying

Here you should use &man.cvsup.1;.

> +		    <filename>distrib/cvsup/sup/README</filename>
> +		    and adding the following files into
> +		    <filename>cvsup/sup/ports-categoryname</filename>:
> +		    <filename>list.cvs</filename> and
> +		    <filename>releases</filename>.</para>
> +		</step>
> +
> +		<step>
> +		  <para>Submit a PR to add the new category both to the
> +		    <ulink=20
> url=3D"../../books/porters-handbook/makefile-categories.html#PORTING-CATE=
GORIES">
> +		    Porter's Handbook</ulink> and to the file
> +		    <filename>www/en/ports/categories</filename>,
> +		    and assign it to <literal>docs</literal>.</para>

If the category is set to "docs" there is no need to reassign it.  It
will automatically be assigned to freebsd-doc.

> +		</step>
> +
> +		<step>
> +		  <para>Submit a PR to request that the web
> +		    page build be updated to reflect the new
> +		    category.  The responsibility for doing this
> +		    is not yet well established.</para>
> +		</step>

Shouldn't this just be a "www" PR?  I'm not really sure what needs to be
changed on the web pages?

Of course the issue with the ports/ pages being based on a INDEX still
exist...

--=20
Simon L. Nielsen
FreeBSD Documentation Team

--OgqxwSJOaUobr8KG
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.3 (FreeBSD)

iD8DBQFAadSah9pcDSc1mlERAtxyAJ4/iyF6Ijqd7HIChWmqG4O7v1s8jgCfXkr/
yRFadwmFGQsIUEEJQpbZA9w=
=6K8P
-----END PGP SIGNATURE-----

--OgqxwSJOaUobr8KG--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040330201210.GA749>