Re: timezone changes break windows and cygwin

Lists: pgsql-hackers
From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: timezone changes break windows and cygwin
Date: 2005-07-05 13:22:43
Message-ID: 42CA89A3.1070104@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers


The latest timezone lib changes seem to have broken Windows and Cygwin
builds comprehensively. I assume that this checkin caused the problem:

http://archives.postgresql.org/pgsql-committers/2005-07/msg00155.php :

Restructure zic #define fprintf checks to use a NO_PGPORT macro instead.

Extracts from buildfarm logs:

Windows:

gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -fno-strict-aliasing -g zic.o ialloc.o scheck.o localtime-zic.o -o zic.exe
zic.o(.text+0xe99): In function `dolink':
C:/msys/1.0/home/pgbf/pgbuildfarm/HEAD/pgsql.8032/src/timezone/zic.c:634: undefined reference to `pgsymlink'
zic.o(.text+0xed5):C:/msys/1.0/home/pgbf/pgbuildfarm/HEAD/pgsql.8032/src/timezone/zic.c:613: undefined reference to `pgunlink'
zic.o(.text+0x36ed): In function `writezone':
C:/msys/1.0/home/pgbf/pgbuildfarm/HEAD/pgsql.8032/src/timezone/zic.c:1580: undefined reference to `pgunlink'
localtime-zic.o(.text+0x14c): In function `tzload':
C:/msys/1.0/home/pgbf/pgbuildfarm/HEAD/pgsql.8032/src/timezone/localtime-zic.c:161: undefined reference to `win32_open'

Cygwin:

gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Wendif-labels -fno-strict-aliasing -g zic.o ialloc.o scheck.o localtime-zic.o -o zic.exe
zic.o(.text+0x86c): In function `dolink':
/home/jim/pgbuildfarm/HEAD/pgsql.7860/src/timezone/zic.c:613: undefined reference to `_pgunlink'
zic.o(.text+0x261a): In function `writezone':
/home/jim/pgbuildfarm/HEAD/pgsql.7860/src/timezone/zic.c:1562: undefined reference to `_pgunlink'
collect2: ld returned 1 exit status

cheers

andrew


From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: timezone changes break windows and cygwin
Date: 2005-07-05 17:24:02
Message-ID: 200507051724.j65HO2l02476@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Andrew Dunstan wrote:
>
> The latest timezone lib changes seem to have broken Windows and Cygwin
> builds comprehensively. I assume that this checkin caused the problem:
>
> http://archives.postgresql.org/pgsql-committers/2005-07/msg00155.php :
>
> Restructure zic #define fprintf checks to use a NO_PGPORT macro instead.

Actually, that patch was to fix my unix build. The big change was that
zic no longer uses pgport routines, and that was done by someone else so
zic would be run on the target matchine for cross-platform builds.

I have applied the attached patch which should fix the problems you saw.
It basically extends NO_PGPORT, which I expected might be required when
I added it.

---------------------------------------------------------------------------
>
>
> Extracts from buildfarm logs:
>
> Windows:
>
> gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -fno-strict-aliasing -g zic.o ialloc.o scheck.o localtime-zic.o -o zic.exe
> zic.o(.text+0xe99): In function `dolink':
> C:/msys/1.0/home/pgbf/pgbuildfarm/HEAD/pgsql.8032/src/timezone/zic.c:634: undefined reference to `pgsymlink'
> zic.o(.text+0xed5):C:/msys/1.0/home/pgbf/pgbuildfarm/HEAD/pgsql.8032/src/timezone/zic.c:613: undefined reference to `pgunlink'
> zic.o(.text+0x36ed): In function `writezone':
> C:/msys/1.0/home/pgbf/pgbuildfarm/HEAD/pgsql.8032/src/timezone/zic.c:1580: undefined reference to `pgunlink'
> localtime-zic.o(.text+0x14c): In function `tzload':
> C:/msys/1.0/home/pgbf/pgbuildfarm/HEAD/pgsql.8032/src/timezone/localtime-zic.c:161: undefined reference to `win32_open'
>
>
> Cygwin:
>
> gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Wendif-labels -fno-strict-aliasing -g zic.o ialloc.o scheck.o localtime-zic.o -o zic.exe
> zic.o(.text+0x86c): In function `dolink':
> /home/jim/pgbuildfarm/HEAD/pgsql.7860/src/timezone/zic.c:613: undefined reference to `_pgunlink'
> zic.o(.text+0x261a): In function `writezone':
> /home/jim/pgbuildfarm/HEAD/pgsql.7860/src/timezone/zic.c:1562: undefined reference to `_pgunlink'
> collect2: ld returned 1 exit status
>
>
> cheers
>
> andrew
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 4: Don't 'kill -9' the postmaster
>

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

Attachment Content-Type Size
unknown_filename text/plain 1.5 KB

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: timezone changes break windows and cygwin
Date: 2005-07-06 02:18:56
Message-ID: 42CB3F90.3010207@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Cygwin seems fixed, but now on my Windows box I get this:

gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -fno-strict-aliasing
-I../../src/include -I./src/include/port/win32 -DEXEC_BACKEND
"-I../../src/include/port/win32" -DBUILDING_DLL -I. -DNO_PGPORT -c -o
zic.o zic.c
zic.c: In function `dolink':
zic.c:634: warning: implicit declaration of function `symlink'
gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -fno-strict-aliasing
-I../../src/include -I./src/include/port/win32 -DEXEC_BACKEND
"-I../../src/include/port/win32" -DBUILDING_DLL -I. -DNO_PGPORT -c -o
ialloc.o ialloc.c
gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -fno-strict-aliasing
-I../../src/include -I./src/include/port/win32 -DEXEC_BACKEND
"-I../../src/include/port/win32" -DBUILDING_DLL -I. -DNO_PGPORT -c -o
scheck.o scheck.c
ln -s localtime.c localtime-zic.c
gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -fno-strict-aliasing
-I../../src/include -I./src/include/port/win32 -DEXEC_BACKEND
"-I../../src/include/port/win32" -DBUILDING_DLL -I. -DNO_PGPORT -c -o
localtime-zic.o localtime-zic.c
gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -fno-strict-aliasing
zic.o ialloc.o scheck.o localtime-zic.o -o zic.exe
zic.o(.text+0xe99):zic.c: undefined reference to `symlink'

cheers

andrew

Bruce Momjian wrote:

>Andrew Dunstan wrote:
>
>
>>The latest timezone lib changes seem to have broken Windows and Cygwin
>>builds comprehensively. I assume that this checkin caused the problem:
>>
>>http://archives.postgresql.org/pgsql-committers/2005-07/msg00155.php :
>>
>>Restructure zic #define fprintf checks to use a NO_PGPORT macro instead.
>>
>>
>
>Actually, that patch was to fix my unix build. The big change was that
>zic no longer uses pgport routines, and that was done by someone else so
>zic would be run on the target matchine for cross-platform builds.
>
>I have applied the attached patch which should fix the problems you saw.
>It basically extends NO_PGPORT, which I expected might be required when
>I added it.
>
>
>
>


From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: timezone changes break windows and cygwin
Date: 2005-07-06 03:07:42
Message-ID: 200507060307.j6637gw06185@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers


OK, now we have a problem. :-(

While using native versions of libc functions rather than our pgport
enhanced versions is OK for zic, the use of symlink is a problem because
there is no native Win32 version.

Looking at zic.c::dolink, it calls symlink() if link() fails. I suppose
we could bring in a file from pgport like we bring files in now from
there for libpq.

Is there a reason we can't still use pgport?

---------------------------------------------------------------------------

Andrew Dunstan wrote:
> Cygwin seems fixed, but now on my Windows box I get this:
>
> gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -fno-strict-aliasing
> -I../../src/include -I./src/include/port/win32 -DEXEC_BACKEND
> "-I../../src/include/port/win32" -DBUILDING_DLL -I. -DNO_PGPORT -c -o
> zic.o zic.c
> zic.c: In function `dolink':
> zic.c:634: warning: implicit declaration of function `symlink'
> gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -fno-strict-aliasing
> -I../../src/include -I./src/include/port/win32 -DEXEC_BACKEND
> "-I../../src/include/port/win32" -DBUILDING_DLL -I. -DNO_PGPORT -c -o
> ialloc.o ialloc.c
> gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -fno-strict-aliasing
> -I../../src/include -I./src/include/port/win32 -DEXEC_BACKEND
> "-I../../src/include/port/win32" -DBUILDING_DLL -I. -DNO_PGPORT -c -o
> scheck.o scheck.c
> ln -s localtime.c localtime-zic.c
> gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -fno-strict-aliasing
> -I../../src/include -I./src/include/port/win32 -DEXEC_BACKEND
> "-I../../src/include/port/win32" -DBUILDING_DLL -I. -DNO_PGPORT -c -o
> localtime-zic.o localtime-zic.c
> gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -fno-strict-aliasing
> zic.o ialloc.o scheck.o localtime-zic.o -o zic.exe
> zic.o(.text+0xe99):zic.c: undefined reference to `symlink'
>
> cheers
>
> andrew
>
> Bruce Momjian wrote:
>
> >Andrew Dunstan wrote:
> >
> >
> >>The latest timezone lib changes seem to have broken Windows and Cygwin
> >>builds comprehensively. I assume that this checkin caused the problem:
> >>
> >>http://archives.postgresql.org/pgsql-committers/2005-07/msg00155.php :
> >>
> >>Restructure zic #define fprintf checks to use a NO_PGPORT macro instead.
> >>
> >>
> >
> >Actually, that patch was to fix my unix build. The big change was that
> >zic no longer uses pgport routines, and that was done by someone else so
> >zic would be run on the target matchine for cross-platform builds.
> >
> >I have applied the attached patch which should fix the problems you saw.
> >It basically extends NO_PGPORT, which I expected might be required when
> >I added it.
> >
> >
> >
> >
>

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073


From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, Peter Eisentraut <peter_e(at)gmx(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: timezone changes break windows and cygwin
Date: 2005-07-06 04:55:45
Message-ID: 21972.1120625745@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> OK, now we have a problem. :-(

No kidding. I said to begin with that this plan to use target-specific
configuration knowledge to build a program executable by the host would
not work. I'm for reverting Peter's initial patch; maybe we can someday
find an answer, but this ain't it.

regards, tom lane


From: "Andrew Dunstan" <andrew(at)dunslane(dot)net>
To: <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: <pgman(at)candle(dot)pha(dot)pa(dot)us>, <andrew(at)dunslane(dot)net>, <peter_e(at)gmx(dot)net>, <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: timezone changes break windows and cygwin
Date: 2005-07-06 09:56:42
Message-ID: 1622.24.211.165.134.1120643802.squirrel@www.dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Tom Lane said:
> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
>> OK, now we have a problem. :-(
>
> No kidding. I said to begin with that this plan to use target-specific
> configuration knowledge to build a program executable by the host would
> not work. I'm for reverting Peter's initial patch; maybe we can
> someday find an answer, but this ain't it.
>

+1.

cheers

andrew


From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, Peter Eisentraut <peter_e(at)gmx(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Magnus Hagander <mha(at)sollentuna(dot)net>
Subject: Re: timezone changes break windows and cygwin
Date: 2005-07-06 14:26:58
Message-ID: 200507061426.j66EQwp14541@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Tom Lane wrote:
> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > OK, now we have a problem. :-(
>
> No kidding. I said to begin with that this plan to use target-specific
> configuration knowledge to build a program executable by the host would
> not work. I'm for reverting Peter's initial patch; maybe we can someday
> find an answer, but this ain't it.

Yea, I knew my original NO_PGPORT wasn't going to be the last, but now
we are stuck.

Add to this something Magnus mentioned that I did not see until just
now. The backend links in timezone/SUBSYS.o, which doesn't use pgport,
so you have pgport versions and native versions of some functions in the
same backend binary. I am not sure that will always work. Add to that,
are those object files fully compatible with the backend?

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073


From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, Peter Eisentraut <peter_e(at)gmx(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Magnus Hagander <mha(at)sollentuna(dot)net>
Subject: Re: timezone changes break windows and cygwin
Date: 2005-07-06 14:29:26
Message-ID: 26037.1120660166@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> Add to this something Magnus mentioned that I did not see until just
> now. The backend links in timezone/SUBSYS.o, which doesn't use pgport,
> so you have pgport versions and native versions of some functions in the
> same backend binary. I am not sure that will always work. Add to that,
> are those object files fully compatible with the backend?

The patch was never supposed to change timezone/SUBSYS.o! Only the zic
executable. If it's having any side effects on what goes into SUBSYS.o,
it's simply wrong.

regards, tom lane


From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, Peter Eisentraut <peter_e(at)gmx(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Magnus Hagander <mha(at)sollentuna(dot)net>
Subject: Re: timezone changes break windows and cygwin
Date: 2005-07-06 14:46:56
Message-ID: 200507061446.j66EkuA18052@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Tom Lane wrote:
> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > Add to this something Magnus mentioned that I did not see until just
> > now. The backend links in timezone/SUBSYS.o, which doesn't use pgport,
> > so you have pgport versions and native versions of some functions in the
> > same backend binary. I am not sure that will always work. Add to that,
> > are those object files fully compatible with the backend?
>
> The patch was never supposed to change timezone/SUBSYS.o! Only the zic
> executable. If it's having any side effects on what goes into SUBSYS.o,
> it's simply wrong.

Well, my NO_PGPORT is affecting timezone/SUBSYS.o because there is no
way to compile zic without removing pgport library symbols from the
object files.

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073


From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, Peter Eisentraut <peter_e(at)gmx(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Magnus Hagander <mha(at)sollentuna(dot)net>
Subject: Re: timezone changes break windows and cygwin
Date: 2005-07-06 15:04:26
Message-ID: 26502.1120662266@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> Tom Lane wrote:
>> The patch was never supposed to change timezone/SUBSYS.o! Only the zic
>> executable. If it's having any side effects on what goes into SUBSYS.o,
>> it's simply wrong.

> Well, my NO_PGPORT is affecting timezone/SUBSYS.o because there is no
> way to compile zic without removing pgport library symbols from the
> object files.

We might have to move zic into a separate directory so that it can be
compiled on its own with its own copies of the relevant .o files.

In the meantime though it's utterly clear that this entire series of
patches is a failed experiment. Please revert the lot.

regards, tom lane


From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, Peter Eisentraut <peter_e(at)gmx(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Magnus Hagander <mha(at)sollentuna(dot)net>
Subject: Re: timezone changes break windows and cygwin
Date: 2005-07-06 15:15:51
Message-ID: 200507061515.j66FFpR04880@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Tom Lane wrote:
> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > Tom Lane wrote:
> >> The patch was never supposed to change timezone/SUBSYS.o! Only the zic
> >> executable. If it's having any side effects on what goes into SUBSYS.o,
> >> it's simply wrong.
>
> > Well, my NO_PGPORT is affecting timezone/SUBSYS.o because there is no
> > way to compile zic without removing pgport library symbols from the
> > object files.
>
> We might have to move zic into a separate directory so that it can be
> compiled on its own with its own copies of the relevant .o files.
>
> In the meantime though it's utterly clear that this entire series of
> patches is a failed experiment. Please revert the lot.

I can pull out NO_PGPORT, but what commit causes the original problem?
Was it only one? (I wasn't watching.)

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073


From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, Peter Eisentraut <peter_e(at)gmx(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Magnus Hagander <mha(at)sollentuna(dot)net>
Subject: Re: timezone changes break windows and cygwin
Date: 2005-07-06 15:26:52
Message-ID: 26720.1120663612@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> Tom Lane wrote:
>> In the meantime though it's utterly clear that this entire series of
>> patches is a failed experiment. Please revert the lot.

> I can pull out NO_PGPORT, but what commit causes the original problem?
> Was it only one? (I wasn't watching.)

This one:

2005-07-03 14:54 petere

* configure, configure.in, doc/src/sgml/installation.sgml,
src/Makefile.global.in, src/timezone/Makefile: Support cross
compilation by compiling "zic" with a native compiler. This relies
on the output of zic being platform independent, but that is
currently the case.

and all the subsequent changes touching src/timezone/

regards, tom lane


From: Peter Eisentraut <peter_e(at)gmx(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, Andrew Dunstan <andrew(at)dunslane(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Magnus Hagander <mha(at)sollentuna(dot)net>
Subject: Re: timezone changes break windows and cygwin
Date: 2005-07-06 19:42:03
Message-ID: 200507062142.04491.peter_e@gmx.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Tom Lane wrote:
> > I can pull out NO_PGPORT, but what commit causes the original
> > problem? Was it only one? (I wasn't watching.)
>
> This one:
>
> 2005-07-03 14:54 petere

Attached is the patch, in case someone wants to revert it. (I'd do it
myself but I suppose this would interfere with rolling back the
subsequent patches.)

I apologize for all the trouble.

--
Peter Eisentraut
http://developer.postgresql.org/~petere/

Attachment Content-Type Size
cross-patch.diff text/x-diff 6.7 KB

From: Peter Eisentraut <peter_e(at)gmx(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, Andrew Dunstan <andrew(at)dunslane(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Magnus Hagander <mha(at)sollentuna(dot)net>
Subject: Re: timezone changes break windows and cygwin
Date: 2005-07-06 19:54:25
Message-ID: 200507062154.26601.peter_e@gmx.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Tom Lane wrote:
> We might have to move zic into a separate directory so that it can be
> compiled on its own with its own copies of the relevant .o files.

Up next: rewriting zic in Perl :-)

No seriously, when the dust has settled I think we should just put in a
note into the makefile (or perhaps at the end of configure) to the
effect "you are cross-compiling, you will have problems here and here"
as I had previously proposed. Those who are just looking for a cross
build of libpq should be content with that.

--
Peter Eisentraut
http://developer.postgresql.org/~petere/


From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andrew Dunstan <andrew(at)dunslane(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Magnus Hagander <mha(at)sollentuna(dot)net>
Subject: Re: timezone changes break windows and cygwin
Date: 2005-07-06 21:06:46
Message-ID: 200507062106.j66L6kA18315@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Peter Eisentraut wrote:
> Tom Lane wrote:
> > > I can pull out NO_PGPORT, but what commit causes the original
> > > problem? Was it only one? (I wasn't watching.)
> >
> > This one:
> >
> > 2005-07-03 14:54 petere
>
> Attached is the patch, in case someone wants to revert it. (I'd do it
> myself but I suppose this would interfere with rolling back the
> subsequent patches.)
>
> I apologize for all the trouble.

I backed it out using CVS because this patch didn't contain the SGML
changes that also needed to be reverted, I think.

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073


From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andrew Dunstan <andrew(at)dunslane(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Magnus Hagander <mha(at)sollentuna(dot)net>
Subject: Re: timezone changes break windows and cygwin
Date: 2005-07-06 21:40:29
Message-ID: 200507062140.j66LeTj28051@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers


OK, I have backed out Peter's change, and the NO_PGPORT workarounds.

---------------------------------------------------------------------------

Peter Eisentraut wrote:
> Tom Lane wrote:
> > We might have to move zic into a separate directory so that it can be
> > compiled on its own with its own copies of the relevant .o files.
>
> Up next: rewriting zic in Perl :-)
>
> No seriously, when the dust has settled I think we should just put in a
> note into the makefile (or perhaps at the end of configure) to the
> effect "you are cross-compiling, you will have problems here and here"
> as I had previously proposed. Those who are just looking for a cross
> build of libpq should be content with that.
>
> --
> Peter Eisentraut
> http://developer.postgresql.org/~petere/
>

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073