Skip Menu |

This queue is for tickets about the Module-Install CPAN distribution.

Report information
The Basics
Id: 62348
Status: new
Priority: 0/
Queue: Module-Install

People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in:
  • 0.82
  • 0.83
  • 0.84
  • 0.85
  • 0.86
  • 0.87
  • 0.88
  • 0.89
  • 0.90
  • 0.91
  • 0.92
  • 0.93
  • 0.94
  • 0.95
  • 0.96
  • 0.97
  • 0.98
  • 0.99
  • 1.00
Fixed in: (no value)



Subject: Module::Install::_cmp is a function, not a method
There are a couple of places where Module::Install::_cmp is called as a method, but it is really a function. The attached patch fixes the problem. Note that the patch triggers a problem in the 25_perl_version_from.t test, which I will deal in another ticket. Regards, Slaven
Subject: 0001-Module-Install-_cmp-is-a-function-not-a-method.patch
From 59a8ba61a473b25fca46c0a9681874bfc0015c95 Mon Sep 17 00:00:00 2001 From: Slaven Rezic <slaven@rezic.de> Date: Thu, 21 Oct 2010 17:38:14 +0200 Subject: [PATCH] Module::Install::_cmp is a function, not a method --- lib/Module/Install/Admin/Metadata.pm | 2 +- lib/Module/Install/Makefile.pm | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/Module/Install/Admin/Metadata.pm b/lib/Module/Install/Admin/Metadata.pm index 8b27dd1..a17eb65 100644 --- a/lib/Module/Install/Admin/Metadata.pm +++ b/lib/Module/Install/Admin/Metadata.pm @@ -141,7 +141,7 @@ sub dump_meta { $meta->{$key} = { map { @$_ } @{ $val->{$key} } }; } - if ( $self->_cmp( $meta->{configure_requires}->{'ExtUtils::MakeMaker'}, '6.36' ) >= 0 ) { + if ( Module::Install::_cmp( $meta->{configure_requires}->{'ExtUtils::MakeMaker'}, '6.36' ) >= 0 ) { # Starting from this version ExtUtils::MakeMaker requires perl 5.6 unless ( $perl_version && $self->perl_version($perl_version) >= 5.006 ) { $meta->{requires}->{perl} = '5.006'; diff --git a/lib/Module/Install/Makefile.pm b/lib/Module/Install/Makefile.pm index 7dc29f2..a9c8130 100644 --- a/lib/Module/Install/Makefile.pm +++ b/lib/Module/Install/Makefile.pm @@ -213,7 +213,7 @@ sub write { # Make sure we have a new enough MakeMaker require ExtUtils::MakeMaker; - if ( $perl_version and $self->_cmp($perl_version, '5.006') >= 0 ) { + if ( $perl_version and Module::Install::_cmp($perl_version, '5.006') >= 0 ) { # MakeMaker can complain about module versions that include # an underscore, even though its own version may contain one! # Hence the funny regexp to get rid of it. See RT #35800 -- 1.7.0.3