pgsql: Add FLOAT4PASSBYVAL/FLOAT8PASSBYVAL to pg_config.h.win32, as a

Started by Tom Laneabout 18 years ago6 messagescomitters
Jump to latest
#1Tom Lane
tgl@sss.pgh.pa.us

Log Message:
-----------
Add FLOAT4PASSBYVAL/FLOAT8PASSBYVAL to pg_config.h.win32, as a stopgap
measure to get the Windows buildfarm members working again. I don't
know if it's worth exposing these as configurables, or exactly how to
do it in the MSVC build system ...

Modified Files:
--------------
pgsql/src/include:
pg_config.h.win32 (r1.51 -> r1.52)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/pg_config.h.win32?r1=1.51&r2=1.52)

#2Andrew Dunstan
andrew@dunslane.net
In reply to: Tom Lane (#1)
Re: pgsql: Add FLOAT4PASSBYVAL/FLOAT8PASSBYVAL to pg_config.h.win32, as a

Tom Lane wrote:

Log Message:
-----------
Add FLOAT4PASSBYVAL/FLOAT8PASSBYVAL to pg_config.h.win32, as a stopgap
measure to get the Windows buildfarm members working again. I don't
know if it's worth exposing these as configurables, or exactly how to
do it in the MSVC build system ...

see src/tools/msvc/Solution.pm

The options are picked up from config.pl (which is the equivalent of
configure invocation options) in the new() subroutine, and the config
files are written accordingly in the GenerateFiles() subroutine.

In general I'm in favor of keeping these as closely aligned as possible.

cheers

andrew

#3Tom Lane
tgl@sss.pgh.pa.us
In reply to: Andrew Dunstan (#2)
Re: pgsql: Add FLOAT4PASSBYVAL/FLOAT8PASSBYVAL to pg_config.h.win32, as a

Andrew Dunstan <andrew@dunslane.net> writes:

Add FLOAT4PASSBYVAL/FLOAT8PASSBYVAL to pg_config.h.win32, as a stopgap
measure to get the Windows buildfarm members working again. I don't
know if it's worth exposing these as configurables, or exactly how to
do it in the MSVC build system ...

see src/tools/msvc/Solution.pm

The options are picked up from config.pl (which is the equivalent of
configure invocation options) in the new() subroutine, and the config
files are written accordingly in the GenerateFiles() subroutine.

In general I'm in favor of keeping these as closely aligned as possible.

Go for it --- I'm not excited about adding stuff to platform-specific
code that I can't test.

regards, tom lane

#4Dave Page
dpage@pgadmin.org
In reply to: Tom Lane (#3)
Re: pgsql: Add FLOAT4PASSBYVAL/FLOAT8PASSBYVAL to pg_config.h.win32, as a

On Mon, Apr 21, 2008 at 4:52 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:

Go for it --- I'm not excited about adding stuff to platform-specific
code that I can't test.

I'm sure we could sort you out with a Windows box if you like.

*ducks*

:-)

--
Dave Page
EnterpriseDB UK: http://www.enterprisedb.com

#5Magnus Hagander
magnus@hagander.net
In reply to: Andrew Dunstan (#2)
Re: pgsql: Add FLOAT4PASSBYVAL/FLOAT8PASSBYVAL to pg_config.h.win32, as a

Andrew Dunstan wrote:

Tom Lane wrote:

Log Message:
-----------
Add FLOAT4PASSBYVAL/FLOAT8PASSBYVAL to pg_config.h.win32, as a
stopgap measure to get the Windows buildfarm members working
again. I don't know if it's worth exposing these as configurables,
or exactly how to do it in the MSVC build system ...

see src/tools/msvc/Solution.pm

The options are picked up from config.pl (which is the equivalent of
configure invocation options) in the new() subroutine, and the config
files are written accordingly in the GenerateFiles() subroutine.

In general I'm in favor of keeping these as closely aligned as
possible.

Agreed. If it's a configurable option in the autoconf system, it should
be in the msvc one. I'll see if I can get this done during the day
today before you guys wake up again ;-)

//Magnus

#6Magnus Hagander
magnus@hagander.net
In reply to: Magnus Hagander (#5)
Re: pgsql: Add FLOAT4PASSBYVAL/FLOAT8PASSBYVAL to pg_config.h.win32, as a

Magnus Hagander wrote:

Andrew Dunstan wrote:

Tom Lane wrote:

Log Message:
-----------
Add FLOAT4PASSBYVAL/FLOAT8PASSBYVAL to pg_config.h.win32, as a
stopgap measure to get the Windows buildfarm members working
again. I don't know if it's worth exposing these as
configurables, or exactly how to do it in the MSVC build
system ...

see src/tools/msvc/Solution.pm

The options are picked up from config.pl (which is the equivalent
of configure invocation options) in the new() subroutine, and the
config files are written accordingly in the GenerateFiles()
subroutine.

In general I'm in favor of keeping these as closely aligned as
possible.

Agreed. If it's a configurable option in the autoconf system, it
should be in the msvc one. I'll see if I can get this done during the
day today before you guys wake up again ;-)

Done.

//Magnus