Revert r45361 for now "(bug 16084) Default memory limit should be increased"
authorBrion Vibber <brion@users.mediawiki.org>
Wed, 7 Jan 2009 01:42:50 +0000 (01:42 +0000)
committerBrion Vibber <brion@users.mediawiki.org>
Wed, 7 Jan 2009 01:42:50 +0000 (01:42 +0000)
The logic is wrong and will fail to raise the limit to the new limit if the server's default is between 20 and 50.
Further, the whole logic is crappy; we should be doing the check at runtime rather than at install time, since the server's settings may change after we install.

config/index.php

index 4159c69..2810561 100644 (file)
@@ -461,8 +461,8 @@ if( empty( $memlimit ) || $memlimit == -1 ) {
                $n = intval( $m[1] * (1024*1024) );
        }
        if( $n < 20*1024*1024 ) {
-               print "Attempting to raise limit to 50M... ";
-               if( false === ini_set( "memory_limit", "50M" ) ) {
+               print "Attempting to raise limit to 20M... ";
+               if( false === ini_set( "memory_limit", "20M" ) ) {
                        print "failed.<br /><b>" . htmlspecialchars( $memlimit ) . " seems too low, installation may fail!</b>";
                } else {
                        $conf->raiseMemory = true;
@@ -1661,7 +1661,7 @@ set_include_path( implode( PATH_SEPARATOR, \$path ) . PATH_SEPARATOR . get_inclu
 require_once( \"\$IP/includes/DefaultSettings.php\" );
 
 # If PHP's memory limit is very low, some operations may fail.
-" . ($conf->raiseMemory ? '' : '# ' ) . "ini_set( 'memory_limit', '50M' );" . "
+" . ($conf->raiseMemory ? '' : '# ' ) . "ini_set( 'memory_limit', '20M' );" . "
 
 if ( \$wgCommandLineMode ) {
        if ( isset( \$_SERVER ) && array_key_exists( 'REQUEST_METHOD', \$_SERVER ) ) {