Skip to contentSkip to conversation

Clarify $LANGUAGE description

PATCH In review Petr Baudis 2 participants 2 messages Dec 4, 2008 – Dec 5, 2008 1 message not in the archive Man-pages: manual pages for Linux -- Sections 2, 3, 4, 5, and 7

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Clarify $LANGUAGE description

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).

man3/setlocale.3 +2 -9
index a5d9896..21eebfb 100644
@@ -26,6 +26,7 @@
 .\" Modified Sun Jul  4 14:52:16 1999 by Bruno Haible (haible-dhnC2lb7rBJy/[email protected])
 .\" Modified Tue Aug 24 17:11:01 1999 by Andries Brouwer ([email protected])
 .\" Modified Tue Feb  6 03:31:55 2001 by Andries Brouwer ([email protected])
+.\" Modified Thu Dec  4 02:34:27 2008 by Petr Baudis ([email protected])
 .\"
 .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"""
man7/locale.7 +5 -1
index 3da0ed7..8236e0f 100644
@@ -24,6 +24,7 @@
 .\" Modified Sun Jun 01 17:16:34 1997 by Jochen Hein
 .\"   <[email protected]>
 .\" Modified Thu Apr 25 00:43:19 2002 by Bruno Haible <[email protected]>
+.\" Modified Thu Dec  4 02:34:27 2008 by Petr Baudis <[email protected]>
 .\"
 .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

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html

A message that is not in the archive (sent off-list or not yet received): [email protected]

#2 lore raw

Hi Petr,

Quoted 11 lines · Petr Baudis

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).

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

Quoted 61 lines
man3/setlocale.3 +2 -9
index a5d9896..21eebfb 100644
@@ -26,6 +26,7 @@
 .\" Modified Sun Jul  4 14:52:16 1999 by Bruno Haible (haible-dhnC2lb7rBJy/[email protected])
 .\" Modified Tue Aug 24 17:11:01 1999 by Andries Brouwer ([email protected])
 .\" Modified Tue Feb  6 03:31:55 2001 by Andries Brouwer ([email protected])
+.\" Modified Thu Dec  4 02:34:27 2008 by Petr Baudis ([email protected])
 .\"
 .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"""
man7/locale.7 +5 -1
index 3da0ed7..8236e0f 100644
@@ -24,6 +24,7 @@
 .\" Modified Sun Jun 01 17:16:34 1997 by Jochen Hein
 .\"   <[email protected]>
 .\" Modified Thu Apr 25 00:43:19 2002 by Bruno Haible <[email protected]>
+.\" Modified Thu Dec  4 02:34:27 2008 by Petr Baudis <[email protected]>
 .\"
 .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
Signature · 9 lines
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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html