Re: [PATCH] Clarify $LANGUAGE description

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

 



Hi Petr,

On Wed, Dec 3, 2008 at 8:53 PM, Petr Baudis <pasky@xxxxxxx> wrote:
> This patch (against the latest git) attempts to clean up the $LANGUAGE
> description, by removing bogus comments from setlocale(3) and expanding
> the mention in locale(7).
>
> Maybe you will decide that a more detailed description should be left
> to the gettext(3) documentation, but I actually care about the invisible
> part of the patch more since the comments have put me off the track
> initially ($LANGUAGE has nothing to do with setlocale(3) and is
> completely isolated to gettext, as obvious from the glibc sources).
>
> Signed-off-by: Petr Baudis <pasky@xxxxxxx>

I dropped your in-source changelog entries, but otherwise applied the
patch as is. It was a good thing to clean out those comments: they've
languished in the source for a decade or more.

Thanks!

Cheers,

Michael

> diff --git a/man3/setlocale.3 b/man3/setlocale.3
> index a5d9896..21eebfb 100644
> --- a/man3/setlocale.3
> +++ b/man3/setlocale.3
> @@ -26,6 +26,7 @@
>  .\" Modified Sun Jul  4 14:52:16 1999 by Bruno Haible (haible@xxxxxxxxxxxxxx)
>  .\" Modified Tue Aug 24 17:11:01 1999 by Andries Brouwer (aeb@xxxxxx)
>  .\" Modified Tue Feb  6 03:31:55 2001 by Andries Brouwer (aeb@xxxxxx)
> +.\" Modified Thu Dec  4 02:34:27 2008 by Petr Baudis (pasky@xxxxxxx)
>  .\"
>  .TH SETLOCALE 3  1999-07-04 "GNU" "Linux Programmer's Manual"
>  .SH NAME
> @@ -88,13 +89,7 @@ is
>  each part of the locale that should be modified is set according to the
>  environment variables.
>  The details are implementation-dependent.
> -For glibc, first
> -.\" [This is false on my system - must check which library versions do this]
> -.\" if
> -.\" .I category
> -.\" is LC_MESSAGES, the environment variable LANGUAGE is inspected,
> -.\" then
> -(regardless of
> +For glibc, first (regardless of
>  .IR category ),
>  the environment variable
>  .B LC_ALL
> @@ -113,8 +108,6 @@ If its value is not a valid locale specification, the locale
>  is unchanged, and
>  .BR setlocale ()
>  returns NULL.
> -.\" The environment variable LANGUAGE may contain several, colon-separated,
> -.\" locale names.
>  .PP
>  The locale
>  .B """C"""
> diff --git a/man7/locale.7 b/man7/locale.7
> index 3da0ed7..8236e0f 100644
> --- a/man7/locale.7
> +++ b/man7/locale.7
> @@ -24,6 +24,7 @@
>  .\" Modified Sun Jun 01 17:16:34 1997 by Jochen Hein
>  .\"   <jochen.hein@xxxxxxxxxxxxxxxxx>
>  .\" Modified Thu Apr 25 00:43:19 2002 by Bruno Haible <bruno@xxxxxxxxx>
> +.\" Modified Thu Dec  4 02:34:27 2008 by Petr Baudis <pasky@xxxxxxx>
>  .\"
>  .TH LOCALE 7  1993-04-24 "Linux" "Linux Programmer's Manual"
>  .SH NAME
> @@ -98,7 +99,10 @@ and
>  functions to ease the use of these information.
>  The GNU gettext family of
>  functions also obey the environment variable
> -.BR LANGUAGE .
> +.BR LANGUAGE
> +(containing a colon-separated list of locales)
> +if the category is set to a valid locale other than
> +.BR """C""" .
>  .TP
>  .B LC_NUMERIC
>  changes the information used by the
>



-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
git://git.kernel.org/pub/scm/docs/man-pages/man-pages.git
man-pages online: http://www.kernel.org/doc/man-pages/online_pages.html
Found a bug? http://www.kernel.org/doc/man-pages/reporting_bugs.html
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Kernel Documentation]     [Netdev]     [Linux Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux