Page MenuHomePhabricator

ExtensionDistributor: package composer dependencies inside tarballs
Closed, ResolvedPublic

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 3:41 AM
bzimport set Reference to bz68940.
bzimport added a subscriber: Unknown Object (MLST).

This should be doable, once I figure out a way to get composer installed via puppet.

Running "composer install" will create a vendor/autoload.php. How does that file get require'd? Is that the user's responsibility to do so? Or the extension's?

https://github.com/wmde/WikimediaBadges/blob/15d4f2fbf78c1bd531b7eb27d7d6df908a8996f9/WikimediaBadges.php#L28 will autoload vendor/ if it's there, while AFAIS, https://github.com/wikimedia/mediawiki-extensions-Translate/blob/master/Translate.php doesn't (picked a random example).

Legoktm updated the task description. (Show Details)
Legoktm added a project: Librarization.
Legoktm set Security to None.

Change 176294 had a related patch set uploaded (by Legoktm):
extdist: clone composer into /srv/composer

https://gerrit.wikimedia.org/r/176294

Patch-For-Review

Change 176294 merged by Yuvipanda:
extdist: clone composer into /srv/composer

https://gerrit.wikimedia.org/r/176294

Change 176631 had a related patch set uploaded (by Legoktm):
extdist: Add composer location to config

https://gerrit.wikimedia.org/r/176631

Patch-For-Review

Change 176633 had a related patch set uploaded (by Legoktm):
Bundle composer-specified dependencies in tarballs

https://gerrit.wikimedia.org/r/176633

Patch-For-Review

Change 176631 merged by Yuvipanda:
extdist: Add composer location to config

https://gerrit.wikimedia.org/r/176631

Change 176633 merged by Legoktm:
Bundle composer-specified dependencies in tarballs

https://gerrit.wikimedia.org/r/176633

Code deployed, I'll regenerate all the tarballs later today.

Legoktm mentioned this in Unknown Object (Diffusion Commit).Dec 10 2014, 8:37 PM
Legoktm mentioned this in Unknown Object (Diffusion Commit).