Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 12 Nov 2003 20:20:21 +0000
From:      Ceri Davies <setantae@submonkey.net>
To:        Wes Peters <wes@softweyr.com>
Cc:        docs@freebsd.org
Subject:   Re: Did I get this right?
Message-ID:  <20031112202021.GK60410@submonkey.net>
In-Reply-To: <200311120953.35785.wes@softweyr.com>
References:  <200311120953.35785.wes@softweyr.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 Wed, Nov 12, 2003 at 09:53:35AM -0800, Wes Peters wrote:
> I added a section about CUPS, mostly cribbed from the pkg-descr, to the=
=20
> handbook printing chapter.  I'm pretty fuzzy with doc tools and such,=20
> and therefore need a thorough review.  Please have at me over the=20
> attached diff, and thanks in advance.  ;^)

Hi Wes,

Thanks for knocking this together, but just to be cruel I'm going to
pick your patch apart - don't let it put you off ;-)

> RCS file: /home/dcvs/doc/en_US.ISO8859-1/books/handbook/printing/chapter.=
sgml,v
> retrieving revision 1.81
> diff -u -w -u -w -r1.81 chapter.sgml
> --- chapter.sgml	5 Nov 2003 10:36:55 -0000	1.81
> +++ chapter.sgml	12 Nov 2003 17:50:17 -0000
> @@ -4585,6 +4585,27 @@
>  	      url=3D"http://www.lprng.org/"></ulink>.</para>;
>  	</listitem>
>        </varlistentry>
> +      <varlistentry>
> +	<term>CUPS</term>
> +
> + 	<indexterm><primary>CUPS</primary></indexterm>	 =20
> +	<listitem>
> +	  <para><application>CUPS</application>, the Common UNIX Printing
> +	    System, provides a portable printing layer for UNIX-based
> +	    operating systems. It has been developed by Easy Software
                              ^
Two spaces here please.

> +	    Products to promote a standard printing solution for all UNIX
                                                                     ^^^^
&unix;

> +            vendors and users.</para>
              ^
One extra space here, please get rid.
There should also be a new line between this <para> and the next.

> +	  <para>CUPS uses the Internet Printing Protocol ("IPP") as the
                                                          ^^^^^
<acronym>IPP</acronym>

> +	    basis for managing print jobs and queues. The Line Printer
                                                     ^
Add another space (last time I'll mention this, but it goes for the next
occurrence too).

> +	    Daemon ("LPD") Server Message Block ("SMB"), and AppSocket
                    ^^^^^                        ^^^^^
<acronym>LPD</acronym>, <acronym>SMB</acronym>

> +	    (a.k.a. JetDirect) protocols are also supported with reduced
> +	    functionality. CUPS adds network printer browsing and
> +	    PostScript Printer Description ("PPD") based printing
                                            ^^^^^
<acronym>PPD</acronym>

> +	    options to support real-world printing under UNIX.</para>
                                                         ^^^^
&unix;

> +	  <para>The main site for <application>CUPS</application> is <ulink
> +	      url=3D"http://www.cups.org/"></ulink>.</para>;
            ^^
Two extra spaces here.

> +	</listitem>
> +      </varlistentry>
        ^
This whitespace looks funny, but I suspect you're just falling in line with
the rest of the file, in which case it's ok.

Thanks again,

Ceri

--=20

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

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

iD8DBQE/spYFocfcwTS3JF8RAlAZAJ4mgJH4/rcC8+w/8eiGqlIuCsGjbwCfZqoB
5q1wHPinCUCL0l9jhJby+BU=
=I9UI
-----END PGP SIGNATURE-----

--OgqxwSJOaUobr8KG--



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