Exit maintenance script with non-zero status if execute() returns false
authorTim Starling <tstarling@wikimedia.org>
Sun, 20 May 2018 14:26:46 +0000 (16:26 +0200)
committerTim Starling <tstarling@wikimedia.org>
Thu, 31 May 2018 06:46:14 +0000 (16:46 +1000)
commit72f2e984992b2beb174b2b91761bd36311f5c260
treef85533caa96a5a764f6bbf367981ba81052a514a
parent51f26185bf9ff8b70ae4db96e6962a6aef0346e5
Exit maintenance script with non-zero status if execute() returns false

There's a whole lot of shutdown code in doMaintenance.php that is
skipped when you use exit() directly. In HHVM by default, destructors
are not even called.

There are a few cases where maintenance scripts want to do something after
doMaintenance.php returns, so returning null should continue to mean no exit.

Change-Id: I0891e2ee3af7ef2c64c03b70edcf9e281ce1e7ba
maintenance/Maintenance.php
maintenance/doMaintenance.php