Fix RebuildLocalisationCache bug from MediaWikiServices
authorBryan Davis <bd808@wikimedia.org>
Mon, 11 Apr 2016 19:11:03 +0000 (13:11 -0600)
committerThcipriani <tcipriani@wikimedia.org>
Mon, 11 Apr 2016 19:16:31 +0000 (19:16 +0000)
Revert changes to rebuildLocalisationCache.php introduced in Ie98bf5a.
The forked children do not need separate service connections and the
reset process fails due to MW_SERVICE_BOOTSTRAP_COMPLETE being defined.

Bug: T132370
Change-Id: I0d7ddcd66e7c6945fabd91fea29a48805bf14793

maintenance/rebuildLocalisationCache.php

index 82b362b..b9c797d 100644 (file)
@@ -29,8 +29,6 @@
  * @ingroup Maintenance
  */
 
-use MediaWiki\MediaWikiServices;
-
 require_once __DIR__ . '/Maintenance.php';
 
 /**
@@ -113,8 +111,9 @@ class RebuildLocalisationCache extends Maintenance {
                        $pid = ( $threads > 1 ) ? pcntl_fork() : -1;
 
                        if ( $pid === 0 ) {
-                               // Reset services, so we don't re-use connections.
-                               MediaWikiServices::resetChildProcessServices();
+                               // Child, reseed because there is no bug in PHP:
+                               // http://bugs.php.net/bug.php?id=42465
+                               mt_srand( getmypid() );
 
                                $this->doRebuild( $codes, $lc, $force );
                                exit( 0 );