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.
Jens
Reply to: