diff options
author | Noah Misch <noah@leadboat.com> | 2017-12-08 18:06:05 -0800 |
---|---|---|
committer | Noah Misch <noah@leadboat.com> | 2017-12-08 18:06:22 -0800 |
commit | 9b5c99790e83ca539932426f41ae6e14b1748e8d (patch) | |
tree | f0e9f7a42a6e40811b6dd4d395c60eabfcdab25f /src/backend/access/gist/gistbuild.c | |
parent | a2c6cf36608e10aa223fef49323b5feba344bfcf (diff) | |
download | postgresql-9b5c99790e83ca539932426f41ae6e14b1748e8d.tar.gz postgresql-9b5c99790e83ca539932426f41ae6e14b1748e8d.zip |
MSVC: Test whether 32-bit Perl needs -D_USE_32BIT_TIME_T.
Commits 5a5c2feca3fd858e70ea348822595547e6fa6c15 and
b5178c5d08ca59e30f9d9428fa6fdb2741794e65 introduced support for modern
MSVC-built, 32-bit Perl, but they broke use of MinGW-built, 32-bit Perl
distributions like Strawberry Perl and modern ActivePerl. Perl has no
robust means to report whether it expects a -D_USE_32BIT_TIME_T ABI, so
test this. Back-patch to 9.3 (all supported versions).
The chief alternative was a heuristic of adding -D_USE_32BIT_TIME_T when
$Config{gccversion} is nonempty. That banks on every gcc-built Perl
using the same ABI. gcc could change its default ABI the way MSVC once
did, and one could build Perl with gcc and the non-default ABI.
The GNU make build system could benefit from a similar test, without
which it does not support MSVC-built Perl. For now, just add a comment.
Most users taking the special step of building Perl with MSVC probably
build PostgreSQL with MSVC.
Discussion: https://postgr.es/m/20171130041441.GA3161526@rfd.leadboat.com
Diffstat (limited to 'src/backend/access/gist/gistbuild.c')
0 files changed, 0 insertions, 0 deletions