[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [PATCH] Some improvements/corrections for release-notes.en.sgml

Hi nori,

On Sun, Aug 12, 2007 at 06:38:12PM +0900, Kobayashi Noritada wrote:
> I've found some obvious mistakes and some nonobvious suspicious points
> in the Etch Release Notes.  Could you please find and apply the

it always the same: many missing tags such as <file>, <package> or
<prgn> ... The problem is that this patch could unfuzzy again a few
languages but maybe someone wants to unfuzzy these and send a patch for
multiple languages? Someone?

As long as we do not update the SGML file for PO based languages (by
calling make update-po; make release-notes.<lang>.sgml) they will stay
at 100%. So no need to worry (except that this patch is not yet applied
to such languages).

> attached patch to make improvements and corrections?  I believe that
> most of the corrections are good, but if the patch does some
> overcorrections, sorry.

Thanks for your patch. There is no reason not to apply it! I also think
that proper tag usage simplifies the task of translators (no need to
lookup a program name in a dictionary, ...)

> --- release-notes.en.sgml.orig	2007-08-11 06:06:34.000000000 +0900
> +++ release-notes.en.sgml	2007-08-12 17:52:31.000000000 +0900
> @@ -716,11 +716,11 @@
>          later do the same in the installed system.
>          To get the installer to load the firmware, one has to have network
>          connectivity while the machine is being installed in order to
> -        download the firmware-qlogic udeb package with wget, install it
> +        download the <package>firmware-qlogic</package> udeb package with wget, install it

wget could be tagged too ...

> -        <p>On a system with no X, no additional aptitude install command should be
> +        <p>On a system with no X, no additional <tt>aptitude install</tt> command should be

"aptitude install" is no command (so <tt> is fine, <prgn> would be
wrong)!? The command is aptitude! But let's ignore this ...

> -          check whether the command <tt/cat /dev/urandom > /dev/dsp/ works for root.</p>
> +          check whether the command <tt>cat /dev/urandom > /dev/dsp</tt> works for root.</p>

Good catch! The first / from /dev closed the <tt> and the last / was
wrong as /dev/dsp is a file not a directory.

Thanks, again.


Reply to: