Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit c343314

Browse files
committed
Further tweaks to compiler flags for PL/Perl on Windows.
It now emerges that we can only rely on Perl to tell us we must use -D_USE_32BIT_TIME_T if it's Perl 5.13.4 or later. For older versions, revert to our previous practice of assuming we need that symbol in all 32-bit Windows builds. This is not ideal, but inquiring into which compiler version Perl was built with seems far too fragile. In any case, we had not previously had complaints about these old Perl versions, so let's assume this is Good Enough. (It's still better than the situation ante commit 5a5c2fe, in that at least the effects are confined to PL/Perl rather than the whole PG build.) Back-patch to all supported versions, like 5a5c2fe and predecessors. Discussion: https://postgr.es/m/CANFyU97OVQ3+Mzfmt3MhuUm5NwPU=-FtbNH5Eb7nZL9ua8=rcA@mail.gmail.com
1 parent 2a028de commit c343314

File tree

1 file changed

+12
-0
lines changed

1 file changed

+12
-0
lines changed

src/tools/msvc/Mkvcbuild.pm

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -529,6 +529,18 @@ sub mkvcbuild
529529
}
530530
}
531531

532+
# Perl versions before 5.13.4 don't provide -D_USE_32BIT_TIME_T
533+
# regardless of how they were built. On 32-bit Windows, assume
534+
# such a version was built with a pre-MSVC-2005 compiler, and
535+
# define the symbol anyway, so that we are compatible if we're
536+
# being built with a later MSVC version.
537+
push(@perl_embed_ccflags, '_USE_32BIT_TIME_T')
538+
if $solution->{platform} eq 'Win32'
539+
&& $Config{PERL_REVISION} == 5
540+
&& ($Config{PERL_VERSION} < 13
541+
|| ( $Config{PERL_VERSION} == 13
542+
&& $Config{PERL_SUBVERSION} < 4));
543+
532544
# Also, a hack to prevent duplicate definitions of uid_t/gid_t
533545
push(@perl_embed_ccflags, 'PLPERL_HAVE_UID_GID');
534546

0 commit comments

Comments
 (0)