From: | "Magnus Hagander" <mha(at)sollentuna(dot)net> |
---|---|
To: | "Tom Lane" <tgl(at)postgresql(dot)org>, <pgsql-committers(at)postgresql(dot)org> |
Subject: | Re: pgsql: Arrange to set the LC_XXX environment variables to match our |
Date: | 2005-12-31 14:35:02 |
Message-ID: | 6BCB9D8A16AC4241919521715F4D8BCE92E968@algol.sollentuna.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
This patch appears to have killed win32 builds that *don't* have NLS
enabled:
gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Winline
-Wdeclaration-after-
statement -Wendif-labels -fno-strict-aliasing -g
-I../../../../src/include -I./s
rc/include/port/win32 -DEXEC_BACKEND -I/f/krb5/inc/krb5
"-I../../../../src/incl
ude/port/win32" -DBUILDING_DLL -c -o pg_locale.o pg_locale.c
pg_locale.c: In function `pg_perm_setlocale':
pg_locale.c:109: error: `LC_MESSAGES' undeclared (first use in this
function)
pg_locale.c:109: error: (Each undeclared identifier is reported only
once
pg_locale.c:109: error: for each function it appears in.)
make: *** [pg_locale.o] Error 1
With NLS enabled, it builds.
Which brings a point - with NLS being so troublesome on win32, perhaps
one of the machines building on the buildfarm could be reconfigured to
not build with NLS?
//Magnus
> -----Original Message-----
> From: pgsql-committers-owner(at)postgresql(dot)org
> [mailto:pgsql-committers-owner(at)postgresql(dot)org] On Behalf Of Tom Lane
> Sent: Thursday, December 29, 2005 12:23 AM
> To: pgsql-committers(at)postgresql(dot)org
> Subject: pgsql: Arrange to set the LC_XXX environment
> variables to match our
>
> Log Message:
> -----------
> Arrange to set the LC_XXX environment variables to match our
> locale setup. This protects against undesired changes in
> locale behavior if someone carelessly does setlocale(LC_ALL,
> "") (and we know who you are, perl guys).
>
> Modified Files:
> --------------
> pgsql/src/backend/access/transam:
> xlog.c (r1.223 -> r1.224)
>
> (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/
> access/transam/xlog.c.diff?r1=1.223&r2=1.224)
> pgsql/src/backend/main:
> main.c (r1.97 -> r1.98)
>
> (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/
> main/main.c.diff?r1=1.97&r2=1.98)
> pgsql/src/backend/utils/adt:
> pg_locale.c (r1.32 -> r1.33)
>
> (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/
> utils/adt/pg_locale.c.diff?r1=1.32&r2=1.33)
> pgsql/src/include/utils:
> pg_locale.h (r1.20 -> r1.21)
>
> (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/
> utils/pg_locale.h.diff?r1=1.20&r2=1.21)
>
> ---------------------------(end of
> broadcast)---------------------------
> TIP 1: if posting/reading through Usenet, please send an appropriate
> subscribe-nomail command to majordomo(at)postgresql(dot)org
> so that your
> message can get through to the mailing list cleanly
>
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2005-12-31 15:13:52 | Re: pgsql: Arrange to set the LC_XXX environment |
Previous Message | Bruce Momjian | 2005-12-30 23:49:48 | pgsql: Rename send_rfq to send_ready_for_query. |