From: | "Magnus Hagander" <mha(at)sollentuna(dot)net> |
---|---|
To: | <pgsql-hackers-win32(at)postgresql(dot)org>, "pgsql-patches" <pgsql-patches(at)postgresql(dot)org> |
Subject: | Win32 & NLS |
Date: | 2004-10-17 15:31:32 |
Message-ID: | 6BCB9D8A16AC4241919521715F4D8BCE475F0D@algol.sollentuna.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers-win32 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 | Content-Type | Size |
---|---|---|
win32nls.patch | application/octet-stream | 2.4 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2004-10-17 15:47:19 | Re: [PATCHES] Win32 & NLS |
Previous Message | Andrew Dunstan | 2004-10-16 19:37:36 | Re: get_progname and .exe suffix |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2004-10-17 15:47:19 | Re: [PATCHES] Win32 & NLS |
Previous Message | Greg Stark | 2004-10-17 05:10:14 | Re: Nearing final release? |