Re: fix for windows breakage in regression script - Mailing list pgsql-patches

From Tom Lane
Subject Re: fix for windows breakage in regression script
Date
Msg-id 29575.1105768910@sss.pgh.pa.us
Whole thread Raw
In response to Re: fix for windows breakage in regression script  ("Andrew Dunstan" <andrew@dunslane.net>)
Responses Re: fix for windows breakage in regression script
List pgsql-patches
"Andrew Dunstan" <andrew@dunslane.net> writes:
> Tom Lane said:
>> [ scratches head... ]  Why isn't the #undef in pg_config_manual.h
>> firing on Cygwin?

> But on Cygwin, WIN32 is only defined if windows.h has been included (See
> previous discussion - I recall advocating NOT using WIN32 as a marker for
> just this reason).

Urgh ... so it's only because windows.h isn't included till later that
it works properly.

I'm not sure that we need the code in pg_config_manual.h anymore anyway
--- the configure test should be covering this.  But just before release
is no time to be fooling with such things.

I did add cygwin to the unix_socket=no case in pg_regress, and I'm
inclined to leave it that way because it's really the minimal change
from the script's previous behavior on cygwin.  Do you see a strong
reason for undoing that?

            regards, tom lane

pgsql-patches by date:

Previous
From: "Andrew Dunstan"
Date:
Subject: Re: fix for windows breakage in regression script
Next
From: Devrim GUNDUZ
Date:
Subject: Turkish translation updates