Page MenuHomePhabricator

Thanks REL1_27 failing due to "Your requirements could not be resolved to an installable set of packages"
Closed, ResolvedPublic

Description

mwext-php70-phan-docker on Thanks REL1_27 is failing with

20:41:23 [181.2MB/1.56s] Your requirements could not be resolved to an installable set of packages.
20:41:23 [181.2MB/1.56s] 
20:41:23   Problem 1
20:41:23     - wikimedia/equivset 1.3.0 requires wikimedia/utfnormal ^1.1.0 || ^2.0.0 -> satisfiable by wikimedia/utfnormal[v1.1.0, v2.0.0] but these conflict with your requirements or minimum-stability.
20:41:23     - wikimedia/equivset 1.2.0 requires wikimedia/utfnormal ^1.1.0 -> satisfiable by wikimedia/utfnormal[v1.1.0] but these conflict with your requirements or minimum-stability.
20:41:23     - wikimedia/equivset 1.1.0 requires wikimedia/utfnormal ^1.1.0 -> satisfiable by wikimedia/utfnormal[v1.1.0] but these conflict with your requirements or minimum-stability.
20:41:23     - wikimedia/equivset 1.0.0 requires wikimedia/utfnormal ^1.1.0 -> satisfiable by wikimedia/utfnormal[v1.1.0] but these conflict with your requirements or minimum-stability.
20:41:23     - mediawiki/anti-spoof dev-master requires wikimedia/equivset ^1.0 -> satisfiable by wikimedia/equivset[1.0.0, 1.1.0, 1.2.0, 1.3.0].
20:41:23     - Installation request for mediawiki/anti-spoof dev-master -> satisfiable by mediawiki/anti-spoof[dev-master].

Event Timeline

Phan has been introduced recently and iirc it requires some configuration files in the repository. @Addshore should we restrict phan to only run on master branch for now?


* Installation request for mediawiki/anti-spoof dev-master

One of the extension composer.json requires the package mediawiki/anti-spoof. That is a leftover from experimenting with composer, that should not be used to express the dependency since that confuses the composer merge plugin. With recent MediaWiki versions that is handled in extension.json.

That is due to AbuseFilter having:

composer.json
{
  "require": {
    "mediawiki/anti-spoof": "dev-master"
  }
}

That should be removed entirely.

For the composer issue, see also the same explanation at T201794#4524998 :]

That is due to AbuseFilter having:

composer.json
{
  "require": {
    "mediawiki/anti-spoof": "dev-master"
  }
}

That should be removed entirely.

@hashar I'm not entirely sure where are you seeing that composer.json file but fwiw the dependency on anti-spoof was removed from AbuseFilter over a year ago https://gerrit.wikimedia.org/r/c/mediawiki/extensions/AbuseFilter/+/383717

That is in the REL1_27 branch. So probably we should remove that require in the branch.

That is due to AbuseFilter having:

composer.json
{
  "require": {
    "mediawiki/anti-spoof": "dev-master"
  }
}

That should be removed entirely.

@hashar I'm not entirely sure where are you seeing that composer.json file but fwiw the dependency on anti-spoof was removed from AbuseFilter over a year ago https://gerrit.wikimedia.org/r/c/mediawiki/extensions/AbuseFilter/+/383717

That commit doesn't remove it ;)

The web just get more confusing as you go back through git blame

https://github.com/wikimedia/mediawiki-extensions-AbuseFilter/commit/e981f23b92fbb25634fb295486ef9aeeaeeb5dd8
https://github.com/wikimedia/mediawiki-extensions-AbuseFilter/commit/8ffebe35020b19b3aad4862c481d08a0a200cece

Based on https://packagist.org/packages/mediawiki/anti-spoof I wonder if we can just do dev-REL1_27

Patch incoming!

Change 467836 had a related patch set uploaded (by Reedy; owner: Reedy):
[mediawiki/extensions/AbuseFilter@REL1_27] Stop making AbuseFilter in REL1_27 track dev-master of AntiSpoof

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

Yup, that works. Really, it should be done in extension.json, but we don't have that functionality in REL1_27 - https://github.com/wikimedia/mediawiki/blob/REL1_27/docs/extension.schema.json but things are less broken (as long as composer isn't trying to install antispoof even though it's already installed...

Still broken, but it's then broken a per T202669: Thanks REL1_30 and REL1_27 failing with session token errors instead

Change 467836 merged by Reedy:
[mediawiki/extensions/AbuseFilter@REL1_27] Don't install AntiSpoof extension via composer

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

Reedy claimed this task.
Reedy removed a project: Patch-For-Review.