On Tue, Jan 13, 2015 at 7:40 AM, Tyler Romeo tylerromeo@gmail.com wrote:
I know we just added some new maintenance scripts for checking things with composer. I’m sure it wouldn’t be that bad having update.php check first and tell the user to run “composer install” before doing update.php.
Kunal made the new "checkComposerLockUpToDate.php" maintenance script to validate $IP/vendor against the $IP/composer.json file. An end user could either add this to their typical workflow before running update.php or we could try to find a reasonable way to integrate the check it performs into the update script. Checking for external dependencies isn't the same thing at all as updating a database schema so I'd lean towards suggesting that the new script be used separately.
On January 13, 2015 at 08:07:34, Marcin Cieslak (saper@saper.info) wrote:
I am kind of late to the party but I have upgraded one of my throaway development wikis with the usual "git remote update && git merge && php maintenance/update.php" process and after the above succeeded I was nevertheless greeted by:
Fatal error: Class 'Cdb\Reader' not found
exception coming out of includes/cache/LocalisationCache.php on line 1263
It seems that I just forgot to update the "vendor" directory (I am somehow reluctant to run composer due to allow_url_fopen=1) requirement
Would that be reasonable to add some basic external libraries checks to update.php to remind users to update those core components prior to accessing the wiki?
Btw. I think UPGRADE doc does not (yet) mention the new process.
I think that Kunal's thinking on this (Composer and UPGRADE) was that when the 1.25 tarballs are released they will likely bundle the required libraries directly and thus use of Composer will not be needed by the end user. There is a sentence in the Git subsection of https://www.mediawiki.org/wiki/Manual:Upgrading mentioning the external library dependency:
If you are upgrading to MediaWiki 1.25 or later, you will also need to install some external libraries. See the documentation on that for more details.
Maybe that needs a bit more emphasis on the wiki page?
Bryan