Re: [ANNOUNCE] mod_perl 2.0.0-RC6

[prev] [thread] [next] [Date index for 2005/05/08]

From: Stas Bekman
Subject: Re: [ANNOUNCE] mod_perl 2.0.0-RC6
Date: 07:16 on 08 May 2005

Tom Williams wrote:
> Stas Bekman wrote:
> 
>> Changes since RC5:
> 
> 
> (snip)
> 
> 
>> improve the diagnostics when detecting mp2 < 1.999022, tell the user
>> which files and/or dirs need to be removed [Stas]
> 
> 
> I'm trying to install mod_perl 2.0.0-RC6 on a RedHat 9 based Linux 
> system that is currently running Apache 2.0.52/mod_perl 2.0.0-RC4 and 
> I'm getting this error:
> 
> [tom@linux mod_perl-2.0.0-RC6]$ perl Makefile.PL 
> MP_APXS=/usr/local/apache-2.0.54/bin/apxs
> Reading Makefile.PL args from @ARGV
>    MP_APXS = /usr/local/apache-2.0.54/bin/apxs
> Argument "1.999_021" isn't numeric in numeric gt (>) at Makefile.PL line 
> 149.
> mod_perl/1.999_021 installation detected...[tom@linux mod_perl-2.0.0-RC6]$
> 
> I've built Apache 2.0.54 using "config.nice" from my Apache 2.0.52 build 
> and that built without problem (as far as I know).
> 
> Any ideas on what could be wrong?

Tom, please try with this patch:

Index: Makefile.PL
===================================================================
--- Makefile.PL (revision 168093)
+++ Makefile.PL (working copy)
@@ -126,8 +126,9 @@
      set_modperl_version();

      if ($old_modperl_version) {
-        $old_modperl_version =~ s/(\d\d\d?)(\d\d)/$1_$2/;
-        my $vstring = "mod_perl/$old_modperl_version";
+        (my $old_modperl_version_str = $old_modperl_version)
+            =~ s/(\d\d\d?)(\d\d)/$1_$2/;
+        my $vstring = "mod_perl/$old_modperl_version_str";
          print "$vstring installation detected...";

          my $prefix;


        -- 
        __________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@xxxxxx.xxx http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com