On 2014-11-24 8:47 PM, Legoktm wrote:
On 11/24/14 7:13 PM, Mark A. Hershberger wrote:
== Changes since 1.24.0-rc.2 ==
* The composer.json file has been renamed to composer.json.sample after
Jamie Thingelstad reported that his composer.json was overwritten by
the tarball.
Is there an associated bug report? We decided to revert a similar
move
in core and this goes against the consensus in code review and the
wikitech discussions that were held months ago (back in May!), so I
strongly recommend reverting this.
First off, this "change"[1] was done outside of git meaning there was no
opportunity for code review and no possibility of a revert.
Adding a composer.json to MediaWiki has been intentionally done (see
Ori's extremely long rationale on [2]) so MediaWiki can properly depend
on external libraries like phpunit, and a bunch more starting with the
1.25 development cycle.
This has been previously discussed on this list (see [3] and [4])
explaining why we chose to have a composer.json in core. Maybe we need
better release notes documenting this switch since 1.23.
I'm frustrated at why this is coming in at the last minute and done in a
seemingly secret manner especially after there have been so many
discussions about this that the "release team" themselves were involved in.
This probably shouldn't be a problem. The real reason we want a
composer.json is for git, to include 3rd party libraries in core.
These libraries are bundled into the tarball so that tarball users don't
need to `composer install` to get MediaWiki working, especially when
some of them don't even have a CLI.
~Daniel Friesen (Dantman, Nadir-Seen-Fire) [
http://danielfriesen.name/]