Win32 & NLS - Mailing list pgsql-patches

From Magnus Hagander
Subject Win32 & NLS
Date
Msg-id 6BCB9D8A16AC4241919521715F4D8BCE475F0D@algol.sollentuna.se
Whole thread Raw
Responses Re: Win32 & NLS
Re: [pgsql-hackers-win32] Win32 & NLS
Re: [pgsql-hackers-win32] Win32 & NLS
List pgsql-patches
Hi!

Working the NLS stuff on win32. Considering I know very little about
this part (don't use it myself, never coded around in it), perhaps
someone else can shed some light?

PostgreSQL responds correctly to whatever the LC_MESSAGES environment
variable is set to upon startup of postgresql - I get my error messages
in swedish, english, german or whatever depending on that.

However, postgresql.conf does not load with the error message:
FATAL:  invalid value for parameter "lc_messages": "EN"

This goes for whatever I set lc_messages to, including all the
combinations that work when set in the environment variable. If I
comment it out completely from the config file, things appear to work
with the locale picked up from the environment.
(The error msg of course only shows up when NLS is enabled in configure)


Some quick tracking-down of this shows that the code on line 80-82 of
pg_locale.c:

    save = setlocale(category, NULL);
    if (!save)
        return NULL;            /* won't happen, we hope
*/

*does* return NULL...

Now, I really don't know anything about the setlocale() integration, but
from the MSDN documentation
(http://msdn.microsoft.com/library/default.asp?url=/library/en-us/vclib/
html/_crt_setlocale.2c_._wsetlocale.asp) doesn't even list LC_MESSAGES
as a valid identifier. My bet is that's why it returns NULL on
LC_MESSAGES.

Also, some googling lead me to this:
http://www.haible.de/bruno/gettext-FAQ.html#windows_woe32

Which appears to suggest that we should change the locale using putenv()
etc, and not using setlocale() at all... Because setlocale() does not
support LC_MESSAGES, probably.

Attached is a patch with adds a environment variable based version of
locale_messages_assign(). It's not a pretty solution, but I think it's
probably necessary.

Comments?


(There is also a patch for initdb required to work on win32 when
compiled with NLS enabled, but this is the start...)


//Magnus

Attachment

pgsql-patches by date:

Previous
From: Reini Urban
Date:
Subject: Re: [CYGWIN] postgresql 8.0 with cygwin - success
Next
From: Peter Eisentraut
Date:
Subject: Re: Win32 & NLS