You are viewing a plain text version of this content. The canonical link for it is here.
Posted to modperl-cvs@perl.apache.org by ap...@apache.org on 2009/11/20 05:05:38 UTC

svn commit: r882407 - /perl/Apache-SizeLimit/trunk/lib/Apache/SizeLimit/Core.pm

Author: aprime
Date: Fri Nov 20 04:05:37 2009
New Revision: 882407

URL: http://svn.apache.org/viewvc?rev=882407&view=rev
Log:
Fixed issue with solaris version numbers being treated as decimal numbers, not
dotted tuples.
Fixes RT #38084.

Modified:
    perl/Apache-SizeLimit/trunk/lib/Apache/SizeLimit/Core.pm

Modified: perl/Apache-SizeLimit/trunk/lib/Apache/SizeLimit/Core.pm
URL: http://svn.apache.org/viewvc/perl/Apache-SizeLimit/trunk/lib/Apache/SizeLimit/Core.pm?rev=882407&r1=882406&r2=882407&view=diff
==============================================================================
--- perl/Apache-SizeLimit/trunk/lib/Apache/SizeLimit/Core.pm (original)
+++ perl/Apache-SizeLimit/trunk/lib/Apache/SizeLimit/Core.pm Fri Nov 20 04:05:37 2009
@@ -131,7 +131,9 @@
 }
 
 BEGIN {
-    if ($Config{'osname'} eq 'solaris' && $Config{'osvers'} >= 2.6 ) {
+    my ($major,$minor) = split(/\./, $Config{'osvers'});
+    if ($Config{'osname'} eq 'solaris' && 
+        (($major > 2) || ($major == 2 && $minor >= 6))) {
         *_platform_check_size   = \&_solaris_2_6_size_check;
         *_platform_getppid = \&_perl_getppid;
     }



Re: svn commit: r882407 - /perl/Apache-SizeLimit/trunk/lib/Apache/SizeLimit/Core.pm

Posted by "Philippe M. Chiasson" <go...@ectoplasm.org>.
On 09-11-19 23:09 , Adam Prime wrote:
> Can someone change this ticket to resolved (or whatever is appropriate)
> 
> https://rt.cpan.org/Ticket/Display.html?id=38084
> 
> I can't.

Done. Thanks!

-- 
Philippe M. Chiasson     GPG: F9BFE0C2480E7680 1AE53631CB32A107 88C3A5A5
http://gozer.ectoplasm.org/       m/gozer\@(apache|cpan|ectoplasm)\.org/


Re: svn commit: r882407 - /perl/Apache-SizeLimit/trunk/lib/Apache/SizeLimit/Core.pm

Posted by Adam Prime <ad...@utoronto.ca>.
Can someone change this ticket to resolved (or whatever is appropriate)

https://rt.cpan.org/Ticket/Display.html?id=38084

I can't.

Adam

aprime@apache.org wrote:
> Author: aprime
> Date: Fri Nov 20 04:05:37 2009
> New Revision: 882407
> 
> URL: http://svn.apache.org/viewvc?rev=882407&view=rev
> Log:
> Fixed issue with solaris version numbers being treated as decimal numbers, not
> dotted tuples.
> Fixes RT #38084.
> 
> Modified:
>     perl/Apache-SizeLimit/trunk/lib/Apache/SizeLimit/Core.pm
> 
> Modified: perl/Apache-SizeLimit/trunk/lib/Apache/SizeLimit/Core.pm
> URL: http://svn.apache.org/viewvc/perl/Apache-SizeLimit/trunk/lib/Apache/SizeLimit/Core.pm?rev=882407&r1=882406&r2=882407&view=diff
> ==============================================================================
> --- perl/Apache-SizeLimit/trunk/lib/Apache/SizeLimit/Core.pm (original)
> +++ perl/Apache-SizeLimit/trunk/lib/Apache/SizeLimit/Core.pm Fri Nov 20 04:05:37 2009
> @@ -131,7 +131,9 @@
>  }
>  
>  BEGIN {
> -    if ($Config{'osname'} eq 'solaris' && $Config{'osvers'} >= 2.6 ) {
> +    my ($major,$minor) = split(/\./, $Config{'osvers'});
> +    if ($Config{'osname'} eq 'solaris' && 
> +        (($major > 2) || ($major == 2 && $minor >= 6))) {
>          *_platform_check_size   = \&_solaris_2_6_size_check;
>          *_platform_getppid = \&_perl_getppid;
>      }
> 
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org