Page MenuHomePhabricator

Multiple LocalSettings.php with single codebase fails in svnImport.php
Closed, DeclinedPublic

Description

Author: david

Description:
My configuration of MediaWiki uses a single codebase with multiple LocalSettings.php files and databases. This is supported by the standard MediaWiki maintenance scripts, but not by your svnImport.php script. You can find more detail on the old request I made against MediaWiki in this resolved bug:

https://bugzilla.wikimedia.org/show_bug.cgi?id=9058

Please add support for "--conf" and "--aconf" parameters to svnImport.php.


Version: unspecified
Severity: enhancement

Details

Reference
bz27541

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:22 PM
bzimport set Reference to bz27541.
bzimport added a subscriber: Unknown Object (MLST).

The usages of setting --conf and --aconf in trunk seem to have disappeared..

So personally, I think this should be more targetted at the base maintenance scripts...

david wrote:

Standard MediaWiki maintenance scripts continue to use "--conf" as noted here:

http://www.mediawiki.org/wiki/Manual:Maintenance_scripts

The use of "--aconf" isn't needed, but is still accepted. I really just need the standard conf parameter in svnImport.php to be honored.

aconf is probably deprecated with the removal of adminsettings...

reedy@ubuntu64-esxi:~/mediawiki/trunk/extensions/CodeReview$ php svnImport.php
Argument <repo> required!

Import revisions to Code Review from a Subversion repo

Usage: php svnImport.php [--conf|--dbpass|--dbuser|--globals|--help|--memory-limit|--precache|--quiet|--server|--wiki] <repo> [start]

--conf: Location of LocalSettings.php, if not default
--dbpass: The password to use for this script
--dbuser: The DB user to use for this script

To me, that would look like it's already there...

david wrote:

Sorry, I guess the aconf failure made it look like a conf failure. This is not a bug.