On 13 Aug 2008, at 14:56, Oleg Kostyuk wrote:
Hello!
I just experienced problems with upgrading Catalyst::Action::REST to
0.63:
.......
*** Module::AutoInstall configuration finished.
ERROR: perl: Version 5.008008 is installed, but we need version >= 5.8
at inc/Module/Install/Makefile.pm line 172.
Following patch fixes this error:
tux# diff -u inc/Module/Install/Makefile.pm.orig inc/Module/Install/
Makefile.pm
--- inc/Module/Install/Makefile.pm.orig 2008-08-13
16:39:22.000000000 +0300
+++ inc/Module/Install/Makefile.pm 2008-08-13
16:47:23.000000000 +0300
@@ -170,7 +170,7 @@
}
if ( my $perl_version = $self->perl_version ) {
- eval "use $perl_version; 1"
+ eval "use v$perl_version; 1"
or die "ERROR: perl: Version $] is installed, "
. "but we need version >= $perl_version";
}
tux#
--
Sincerely yours,
Oleg Kostyuk (CUB-UANIC)
Kudos for the effort of supplying a patch, but everything in the inc
directory is auto generated by Module::Install.
The problem infact lies in Module::Install::Metadata and its
perl_version fuction
$version =~ s/^(\d+)\.(\d+)\.(\d+)$/sprintf("%d.%03d%03d",
$1,$2,$3)/e;
needs to be
$version =~ s/^(\d+)\.(\d+)(?:\.(\d+))?$/sprintf("%d.%03d%03d",
$1,$2,$3 || 0)/e;
I suspect.
Can you please turn this into a proper diff and submit it as a bug to [EMAIL PROTECTED]
then let us know the ticket id so we can chase it up.
-ash
_______________________________________________
List: Catalyst@lists.scsys.co.uk
Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst
Searchable archive: http://www.mail-archive.com/catalyst@lists.scsys.co.uk/
Dev site: http://dev.catalyst.perl.org/
