possible bug windows setup

Lists: pgsql-bugspgsql-hackers
From: "Gevik Babakhani" <pgdev(at)xs4all(dot)nl>
To: <pgsql-bugs(at)postgresql(dot)org>
Subject: possible bug windows setup
Date: 2008-02-06 13:59:31
Message-ID: 000001c868c8$7f5bce10$0a01a8c0@gevmus
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-bugs pgsql-hackers

I might be very wrong, but when I try to install 8.3 on Windows with NLS
options selected, no share/locale files are installed. could someone please
test or confirm this?

Regards,
Gevik.


From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Gevik Babakhani <pgdev(at)xs4all(dot)nl>
Cc: pgsql-bugs(at)postgresql(dot)org, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: possible bug windows setup
Date: 2008-02-07 10:07:06
Message-ID: 20080207100706.GF12085@svr2.hagander.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-bugs pgsql-hackers

On Wed, Feb 06, 2008 at 02:59:31PM +0100, Gevik Babakhani wrote:
> I might be very wrong, but when I try to install 8.3 on Windows with NLS
> options selected, no share/locale files are installed. could someone please
> test or confirm this?

Yes, it's broken. It seems the change in Install.pm rev 1.20 to use
File::Find instead of external dir broke this, and was never tested at all
:-(

I know I build without NLS enabled, and it seems so does everybody else who
regularly builds the msvc stuff.

There is no testing at all of the NLS stuff in the regression tests.
Perhaps it would be a good idea to do that? Could be somethign as simple as
launching psql in a way tha tgenerates a syntax error and make sure i
matches a proper translation?

(and yes, I'm working on a patch for the actual issue)

//Magnus


From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Magnus Hagander <magnus(at)hagander(dot)net>
Cc: Gevik Babakhani <pgdev(at)xs4all(dot)nl>, pgsql-bugs(at)postgresql(dot)org, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [HACKERS] possible bug windows setup
Date: 2008-02-07 11:12:17
Message-ID: 47AAE791.1060207@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-bugs pgsql-hackers

Magnus Hagander wrote:
> On Wed, Feb 06, 2008 at 02:59:31PM +0100, Gevik Babakhani wrote:
>
>> I might be very wrong, but when I try to install 8.3 on Windows with NLS
>> options selected, no share/locale files are installed. could someone please
>> test or confirm this?
>>
>
> Yes, it's broken. It seems the change in Install.pm rev 1.20 to use
> File::Find instead of external dir broke this, and was never tested at all
> :-(
>
> I know I build without NLS enabled, and it seems so does everybody else who
> regularly builds the msvc stuff.
>
> There is no testing at all of the NLS stuff in the regression tests.
> Perhaps it would be a good idea to do that? Could be somethign as simple as
> launching psql in a way tha tgenerates a syntax error and make sure i
> matches a proper translation?
>
>
> (and yes, I'm working on a patch for the actual issue)
>
>
>

Oops, My bad!

I think if we just change the forward slashes to backslashes at the top[
of that loop it should work, but haven't had time to test.

cheers

andrew


From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: Gevik Babakhani <pgdev(at)xs4all(dot)nl>, pgsql-bugs(at)postgresql(dot)org, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [HACKERS] possible bug windows setup
Date: 2008-02-07 11:16:09
Message-ID: 20080207111609.GJ12085@svr2.hagander.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-bugs pgsql-hackers

On Thu, Feb 07, 2008 at 06:12:17AM -0500, Andrew Dunstan wrote:
>
>
> Magnus Hagander wrote:
> >On Wed, Feb 06, 2008 at 02:59:31PM +0100, Gevik Babakhani wrote:
> >
> >>I might be very wrong, but when I try to install 8.3 on Windows with NLS
> >>options selected, no share/locale files are installed. could someone
> >>please
> >>test or confirm this?
> >>
> >
> >Yes, it's broken. It seems the change in Install.pm rev 1.20 to use
> >File::Find instead of external dir broke this, and was never tested at all
> >:-(
> >
> >I know I build without NLS enabled, and it seems so does everybody else who
> >regularly builds the msvc stuff.
> >
> >There is no testing at all of the NLS stuff in the regression tests.
> >Perhaps it would be a good idea to do that? Could be somethign as simple as
> >launching psql in a way tha tgenerates a syntax error and make sure i
> >matches a proper translation?
> >
> >
> >(and yes, I'm working on a patch for the actual issue)
> >
> >
> >
>
> Oops, My bad!
>
> I think if we just change the forward slashes to backslashes at the top[
> of that loop it should work, but haven't had time to test.

No, it'sa bit more than that, but that's one of the changes :)

I have a patch working for me, I've sent it over to Gevik for testing in
his environment. Attached here if somebody else wants to play.

//Magnus

Attachment Content-Type Size
install.patch text/plain 2.2 KB

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Magnus Hagander <magnus(at)hagander(dot)net>
Cc: Gevik Babakhani <pgdev(at)xs4all(dot)nl>, pgsql-bugs(at)postgresql(dot)org, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [HACKERS] possible bug windows setup
Date: 2008-02-07 11:32:08
Message-ID: 47AAEC38.5050505@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-bugs pgsql-hackers

Magnus Hagander wrote:
>
> I have a patch working for me, I've sent it over to Gevik for testing in
> his environment. Attached here if somebody else wants to play.
>
>

Looks OK.

cheers

andrew