Andrew Dunstan <andrew.dunstan@2ndquadrant.com> writes:
> On 07/13/2017 08:08 AM, Ashutosh Sharma wrote:
>> - dVAR; dXSBOOTARGSAPIVERCHK;
>> + dVAR; dXSBOOTARGSNOVERCHK;
> Good job hunting this down!
> One suggestion I saw in a little googling was that we add this to the XS
> file after the inclusion of XSUB.h:
> #undef dXSBOOTARGSAPIVERCHK
> #define dXSBOOTARGSAPIVERCHK dXSBOOTARGSNOVERCHK
I don't see anything even vaguely like that in the Util.c file generated
by Perl 5.10.1, which is what I've got on my RHEL machine.
What I do notice is this in Util.xs:
VERSIONCHECK: DISABLE
which leads immediately to two questions:
1. Why is your version of xsubpp apparently ignoring this directive
and generating a version check anyway?
2. Why do we have this directive in the first place? It does not seem
to me like a terribly great idea to ignore low-level version mismatches.
In the same vein, I'm suspicious of proposals to "fix" this problem
by removing the version check, which seems to be where Ashutosh
is headed. In the long run that seems certain to cause huge headaches.
regards, tom lane