Re: compiler warnings on the buildfarm - Mailing list pgsql-hackers

From Tom Lane
Subject Re: compiler warnings on the buildfarm
Date
Msg-id 10521.1184273007@sss.pgh.pa.us
Whole thread Raw
In response to Re: compiler warnings on the buildfarm  (Magnus Hagander <magnus@hagander.net>)
Responses Re: compiler warnings on the buildfarm
List pgsql-hackers
Magnus Hagander <magnus@hagander.net> writes:
> Tom Lane wrote:
>> Yeah, this looks like a good list.  I can't readily check the ones from
>> "eel" as they appear to be in Windows-specific code; anyone else want to
>> fix those?

> The pg_ctl one is a windows one, I'll deal with that one.

> The dirmod one doesn't appear on win32, only cygwin. I don't have a
> cygwin to check that against, so I'll have to pass on that one.

Eyeing the code, it looks like the issue is that port.h declares
pgsymlink if#if defined(WIN32) && !defined(__CYGWIN__)
while dirmod.c defines it if#ifdef WIN32

So this seems like an actual bug, at least to the extent that pgsymlink
is being compiled into code but not used on Cygwin.  But more to the
point, maybe port.h is wrong and we should be using pgsymlink on Cygwin?
In any case these two files need to be put into sync.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: 2PC-induced lockup
Next
From: Magnus Hagander
Date:
Subject: Re: compiler warnings on the buildfarm