Last modified: 2011-06-30 17:16:14 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 T31588, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 29588 - Install maintenance script needs to provide appropriate exit status in ALL cases
Install maintenance script needs to provide appropriate exit status in ALL cases
Status: NEW
Product: MediaWiki
Classification: Unclassified
Maintenance scripts (Other open bugs)
1.20.x
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-06-26 00:32 UTC by Edward Z. Yang
Modified: 2011-06-30 17:16 UTC (History)
2 users (show)

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


Attachments

Description Edward Z. Yang 2011-06-26 00:32:25 UTC
Something of a follow up to Bug 17948. We appear to offer an exit code 1 when --help is called, but in no other circumstances (for example, if we fail to connect to the database.)

There might be other maintenance scripts that have grown since 17948 was resolved, which don't have proper exit codes either, but install.php should be a priority, since it's more likely to be scripted with.
Comment 1 Edward Z. Yang 2011-06-26 00:45:16 UTC
This only appears to be a problem for certain types of errors, not all of them. Here is the one particular example I ran into, but there may be more. Is there a reason why support here is so patchy?

ezyang@javelin:~/Dev/phase3$ php maintenance/install.php --dbserver localhost.asdf foo bar
PHP 5.3.3-1ubuntu9.5 is installed.
Warning: Could not find eAccelerator, APC, XCache or WinCache.
Object caching is not enabled.
Found ImageMagick: /usr/bin/convert.
Image thumbnailing will be enabled if you enable uploads.
Using server name "http://localhost".
Warning: The intl PECL extension is not available to handle Unicode normalization, falling back to slow pure-PHP implementation.
If you run a high-traffic site, you should read a little on Unicode normalization.
The environment has been checked.
You can install MediaWiki.
Setting up database
DB connection error: Unknown MySQL server host 'localhost.asdf' (1) (localhost.asdf).
Check the host, username and password and try again.

ezyang@javelin:~/Dev/phase3$ echo $?
0
Comment 2 Mark A. Hershberger 2011-06-30 14:53:50 UTC
(In reply to comment #1)
> Is there
> a reason why support here is so patchy?

The CLI installer has not been a major focus for work.  I needed it for a specific case (to run on the test server), but after that I wasn't too concerned with it.

That said, I would like to see improvement on it.  Do you want to take it over?
Comment 3 Chad H. 2011-06-30 17:16:14 UTC
I kind of wish we had dropped the CLI installer from REL1_17...

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


Navigation
Links