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.
-- Legoktm
[1]
https://gerrit.wikimedia.org/r/#/c/175453/
[2]
https://gerrit.wikimedia.org/r/#/c/132788/
[3]
https://www.mail-archive.com/wikitech-l@lists.wikimedia.org/msg76217.html
[4]
https://www.mail-archive.com/wikitech-l@lists.wikimedia.org/msg76250.html