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/

Reply via email to