Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the XSLoader CPAN distribution.

Report information
The Basics
Id: 61332
Status: resolved
Priority: 0/
Queue: XSLoader

People
Owner: Nobody in particular
Requestors: w.phillip.moore@gmail.com (no email address)
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in: 0.10
Fixed in: 0.14



Subject: Test suite fails on perl5.12 due to new numeric value warning
While rebuilding a number of CPAN modules for perl5.12.2, I discovered that XSLoader 0.10 fails to pass it's test suite, with errors like: Invalid version format (non-numeric data) at /efs/dist/perl5/core/5.12.2-1.001- build004/.exec/x86-64.rhel.5/lib/perl5/DynaLoader.pm line 223. The attached patch simply changes the non-numeric fake value "qunckkk" to "9999.9999", and the resulting code passes all tests for both perl5.10.1 and 5.12.2, at least on CentOS (although there's no reason to assume this won't work everywhere).
Subject: XSLoader-0.10-non-numeric-test-failure.patch
diff -rc XSLoader-0.10/t/XSLoader.t XSLoader-0.10-non-numeric-test-failure/t/XSLoader.t *** XSLoader-0.10/t/XSLoader.t 2008-09-23 00:02:44.000000000 -0700 --- XSLoader-0.10-non-numeric-test-failure/t/XSLoader.t 2010-09-14 11:31:17.000000000 -0700 *************** *** 72,81 **** SKIP: { skip "$module not available", 4 if $extensions !~ /\b$module\b/; ! eval qq{ package $module; XSLoader::load('$module', "qunckkk"); }; ! like( $@, "/^$module object version \\S+ does not match bootstrap parameter (?:qunckkk|0)/", "calling XSLoader::load() with a XS module and an incorrect version" ); ! like( $warnings, "/^\$|^Version string 'qunckkk' contains invalid data; ignoring: 'qunckkk'/", "in Perl 5.10, DynaLoader warns about the incorrect version string" ); eval qq{ package $module; XSLoader::load('$module'); }; --- 72,81 ---- SKIP: { skip "$module not available", 4 if $extensions !~ /\b$module\b/; ! eval qq{ package $module; XSLoader::load('$module', "9999.9999"); }; ! like( $@, "/^$module object version \\S+ does not match bootstrap parameter (?:9999.9999|0)/", "calling XSLoader::load() with a XS module and an incorrect version" ); ! like( $warnings, "/^\$|^Version string '9999.9999' contains invalid data; ignoring: '9999.9999'/", "in Perl 5.10, DynaLoader warns about the incorrect version string" ); eval qq{ package $module; XSLoader::load('$module'); }; Only in XSLoader-0.10-non-numeric-test-failure/t: XSLoader.t~
CC: Mailing list Perl5 <perl5-porters [...] perl.org>
Subject: Re: [rt.cpan.org #61332] Test suite fails on perl5.12 due to new numeric value warning
Date: Tue, 14 Sep 2010 16:26:22 -0700
To: Phillip Moore via RT <bug-XSLoader [...] rt.cpan.org>
From: Ilya Zakharevich <nospam-abuse [...] ilyaz.org>
On Tue, Sep 14, 2010 at 02:42:09PM -0400, Phillip Moore via RT wrote: Show quoted text
> Tue Sep 14 14:42:09 2010: Request 61332 was acted upon. > Transaction: Ticket created by WPMOORE > Queue: XSLoader > Subject: Test suite fails on perl5.12 due to new numeric value warning > Broken in: 0.10 > Severity: Critical > Owner: Nobody > Requestors: w.phillip.moore@gmail.com > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=61332 > > > > While rebuilding a number of CPAN modules for perl5.12.2, I discovered that XSLoader 0.10 fails > to pass it's test suite, with errors like:
And what version is shipped with Perl? If >0.10, why do you test 0.10? Show quoted text
> Invalid version format (non-numeric data) at /efs/dist/perl5/core/5.12.2-1.001- > build004/.exec/x86-64.rhel.5/lib/perl5/DynaLoader.pm line 223.
Show quoted text
> The attached patch simply changes the non-numeric fake value "qunckkk" to "9999.9999", and > the resulting code passes all tests for both perl5.10.1 and 5.12.2, at least on CentOS (although > there's no reason to assume this won't work everywhere).
I do not think this is my code. I have no idea what it is trying to test, so do not know whether this "fix" is appropriate. CC to p5p... Thanks, Ilya
On Tue Sep 14 22:52:36 2010, nospam-abuse@ilyaz.org wrote: Show quoted text
> I do not think this is my code. I have no idea what it is trying to > test, so do not know whether this "fix" is appropriate. > > CC to p5p...
Any word on what the issue is or how to resolve it?
CC: undisclosed-recipients: ;
Subject: Re: [rt.cpan.org #61332] Test suite fails on perl5.12 due to new numeric value warning
Date: Tue, 25 Jan 2011 13:07:59 -0800
To: Mike Doherty via RT <bug-XSLoader [...] rt.cpan.org>
From: Ilya Zakharevich <nospam-abuse [...] ilyaz.org>
On Tue, Jan 25, 2011 at 01:23:47PM -0500, Mike Doherty via RT wrote: Show quoted text
> Queue: XSLoader > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=61332 > > > On Tue Sep 14 22:52:36 2010, nospam-abuse@ilyaz.org wrote:
> > I do not think this is my code. I have no idea what it is trying to > > test, so do not know whether this "fix" is appropriate. > > > > CC to p5p...
> > Any word on what the issue is or how to resolve it?
As I said, I think you are banging at a wrong door. Ilya
Subject: Re: [rt.cpan.org #61332] Test suite fails on perl5.12 due to new numeric value warning
Date: Wed, 26 Jan 2011 09:04:27 +0100
To: bug-XSLoader [...] rt.cpan.org
From: Sébastien Aperghis-Tramoni <saper [...] cpan.org>
Mike Doherty wrote via RT: Show quoted text
> On Tue Sep 14 22:52:36 2010, nospam-abuse@ilyaz.org wrote:
>> I do not think this is my code. I have no idea what it is trying to >> test, so do not know whether this "fix" is appropriate. >> >> CC to p5p...
> > Any word on what the issue is or how to resolve it?
Apologies, I should have answered earlier. I'll fix these tests ASAP. -- Sébastien Aperghis-Tramoni Close the world, txEn eht nepO.
Subject: Re: [rt.cpan.org #61332] Test suite fails on perl5.12 due to new numeric value warning
Date: Sat, 16 Apr 2011 14:37:56 +0200
To: bug-XSLoader [...] rt.cpan.org
From: Sébastien Aperghis-Tramoni <saper [...] cpan.org>
Sébastien Aperghis-Tramoni wrote via RT: Show quoted text
> Mike Doherty wrote via RT: >
>> On Tue Sep 14 22:52:36 2010, nospam-abuse@ilyaz.org wrote:
>>> I do not think this is my code. I have no idea what it is trying to >>> test, so do not know whether this "fix" is appropriate. >>> >>> CC to p5p...
>> >> Any word on what the issue is or how to resolve it?
> > Apologies, I should have answered earlier. > I'll fix these tests ASAP.
This problem should be fixed with the release 0.14, just uploaded on the CPAN, sponsored by the Perl QA Hackathon 2011 in Amsterdam. Again, my apologies for being late. -- Sébastien Aperghis-Tramoni Close the world, txEn eht nepO.