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/] _______________________________________________ Wikitech-l mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/wikitech-l
