Re: pgsql: Prevent infinity and NaN in jsonb/plperl transform - Mailing list pgsql-committers

From Tom Lane
Subject Re: pgsql: Prevent infinity and NaN in jsonb/plperl transform
Date
Msg-id 21194.1525110966@sss.pgh.pa.us
Whole thread Raw
In response to pgsql: Prevent infinity and NaN in jsonb/plperl transform  (Peter Eisentraut <peter_e@gmx.net>)
Responses Re: pgsql: Prevent infinity and NaN in jsonb/plperl transform  (Peter Eisentraut <peter.eisentraut@2ndquadrant.com>)
List pgsql-committers
Peter Eisentraut <peter_e@gmx.net> writes:
> Prevent infinity and NaN in jsonb/plperl transform

So this isn't working on Windows:

contrib/jsonb_plperl/jsonb_plperl.c(226): warning C4013: 'isnan' undefined; assuming extern returning int
[c:\pgbuildfarm\pgbuildroot\HEAD\pgsql.build\jsonb_plperl.vcxproj]
...
.\Release\jsonb_plperl\jsonb_plperl.dll : fatal error LNK1120: 1 unresolved externals
[c:\pgbuildfarm\pgbuildroot\HEAD\pgsql.build\jsonb_plperl.vcxproj]

The reason seems to be this kluge in plperl.h:

/* stop perl headers from hijacking stdio and other stuff on Windows */
#ifdef WIN32
#define WIN32IO_IS_STDIO
/*
 * isnan is defined in both the perl and mingw headers. We don't use it,
 * so this just clears up the compile warning.
 */
#ifdef isnan
#undef isnan
#endif
#endif                            /* WIN32 */

Looks like the half-life of that hack just expired.  What shall
we do about it?

If you don't have an immediate solution, please revert this commit
temporarily, as it's preventing me from making any further progress
on the reattach-failure investigation.

            regards, tom lane


pgsql-committers by date:

Previous
From: Peter Eisentraut
Date:
Subject: pgsql: Don't do logical replication of TRUNCATE of zero tables
Next
From: Peter Eisentraut
Date:
Subject: pgsql: Write error messages about duplicate OIDs to stderr