Re: pgsql: Arrange to set the LC_XXX environment variables to match our

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
>

Responses

Browse pgsql-committers by date

  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.