Page MenuHomePhabricator

SpamBlacklist : set extension name in composer.json
Open, Needs TriagePublic

Description

There is a composer.json file in extension, but the name and type of extension are not filled.

I add them to enable to install extension using composer

Event Timeline

Pyro853 created this task.Mar 29 2018, 9:19 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 29 2018, 9:19 AM

Change 422901 had a related patch set uploaded (by Pyro; owner: Pierre Boutet):
[mediawiki/extensions/SpamBlacklist@master] add extension name in composer.json

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

Change 422901 merged by jenkins-bot:
[mediawiki/extensions/SpamBlacklist@master] add extension name in composer.json

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

Pyro853 closed this task as Resolved.Mar 29 2018, 11:28 AM

See T467 or T61872#1361864 and it links why extensions does not have the name in general. composer.json is mostly used to provide build tools or dependency, but not to be a dependency. That just as hint, I do not want to start a new discussion here.

Legoktm changed the task status from Resolved to Declined.Mar 29 2018, 6:23 PM
Legoktm added subscribers: thiemowmde, Legoktm.

I reverted the patch in Gerrit - it's intentional that this extension isn't installable via composer. As per T467, there's no plan to make more extensions installable via composer right now.

I am sorry about this decision, this patch doesn't enable autoloading with composer.
In T467, I see we decide to not " use composer to register components", but it is possible to install it with composer (i mean here just download the extension) whitout register the extension.

This is done for example in T188188

I made a few research, i didn't found any place where it is told about never user composer to install extension, whereas i found many explanation about how to use composer. Can you give me a link where there is a summary which support your point ?

thiemowmde reopened this task as Open.Mar 31 2018, 5:03 PM

As per T467, there's no plan to make more extensions installable via composer right now.

That's ridiculous. T467 does not forbid extensions to be installable via Composer. If you don't want to use Composer, just don't use it. What's the benefit of actively disallowing others to use it?

Change 423612 had a related patch set uploaded (by Pyro; owner: Pyro):
[mediawiki/extensions/SpamBlacklist@master] Revert "Revert "add extension name in composer.json""

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

Hi !
Could we move forward on this ?
@Legoktm : could you answer about lasts comments above ?

The task has been re-opened, but it is blocked now !