Skip Menu |

This queue is for tickets about the CPANPLUS CPAN distribution.

Report information
The Basics
Id: 56774
Status: resolved
Worked: 10 min
Priority: 0/
Queue: CPANPLUS

People
Owner: BINGOS [...] cpan.org
Requestors: jquelin [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: "cpanp o" fails on invalid version format
$ cpanp o [MSG] Checking if source files are up to date [MSG] Retrieving /home/jquelin/.cpanplus/sourcefiles.s2.21.c0.9003.stored [ERROR] Invalid version format (non-numeric data) at /usr/lib/perl5/vendor_perl/5.10.1/Module/Load/Conditional.pm line 301.
On Wed Apr 21 04:56:26 2010, JQUELIN wrote: Show quoted text
> $ cpanp o > [MSG] Checking if source files are up to date > [MSG] Retrieving /home/jquelin/.cpanplus/sourcefiles.s2.21.c0.9003.stored > [ERROR] Invalid version format (non-numeric data) at > /usr/lib/perl5/vendor_perl/5.10.1/Module/Load/Conditional.pm line 301.
the bug seems to be caused by latest version.pm (opening ticket rt#56856) it would be good nevertheless to trap those errors.
attached patch fixes the problem: --- Conditional.pm.orig 2010-07-24 15:40:38.644493259 +0200 +++ Conditional.pm 2010-07-24 15:40:18.538243306 +0200 @@ -298,10 +298,12 @@ ### Update from JPeacock: apparently qv() and version->new ### are different things, and we *must* use version->new ### here, or things like #30056 might start happening + eval { $href->{uptodate} = version->new( $args->{version} ) <= version->new( $href->{version} ) ? 1 : 0; + }; $href->{uptodate} = 0 if $@; } if ( $DEPRECATED and version->new($]) >= version->new('5.011') ) {
Thank you, according to our records this issue has been resolved by releases of both CPANPLUS and Module::Load::Conditional and updates to perl core.