Last modified: 2011-02-18 19:40:12 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T29541, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 27541 - Multiple LocalSettings.php with single codebase fails in svnImport.php
Multiple LocalSettings.php with single codebase fails in svnImport.php
Status: RESOLVED WORKSFORME
Product: MediaWiki extensions
Classification: Unclassified
CodeReview (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-02-18 19:09 UTC by David Multer
Modified: 2011-02-18 19:40 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description David Multer 2011-02-18 19:09:24 UTC
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.
Comment 1 Sam Reed (reedy) 2011-02-18 19:13:51 UTC
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...
Comment 2 David Multer 2011-02-18 19:23:00 UTC
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.
Comment 3 Sam Reed (reedy) 2011-02-18 19:28:16 UTC
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...
Comment 4 David Multer 2011-02-18 19:40:12 UTC
Sorry, I guess the aconf failure made it look like a conf failure. This is not a bug.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links