1
0
Fork 0

Fix lock file verification to take into account root provider/replacers and output mismatches there more clearly, fixes #11458 (#11475)

pull/11478/head
Jordi Boggiano 2023-05-23 23:15:07 +02:00 committed by GitHub
parent 9d965b9c65
commit 23654389da
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 35 additions and 4 deletions

View File

@ -18,6 +18,7 @@ use Composer\Pcre\Preg;
use Composer\Repository\InstalledRepository; use Composer\Repository\InstalledRepository;
use Composer\Repository\LockArrayRepository; use Composer\Repository\LockArrayRepository;
use Composer\Repository\PlatformRepository; use Composer\Repository\PlatformRepository;
use Composer\Repository\RootPackageRepository;
use Composer\Util\ProcessExecutor; use Composer\Util\ProcessExecutor;
use Composer\Package\Dumper\ArrayDumper; use Composer\Package\Dumper\ArrayDumper;
use Composer\Package\Loader\ArrayLoader; use Composer\Package\Loader\ArrayLoader;
@ -514,9 +515,10 @@ class Locker
if ($includeDev === true) { if ($includeDev === true) {
$sets[] = ['repo' => $this->getLockedRepository(true), 'method' => 'getDevRequires', 'description' => 'Required (in require-dev)']; $sets[] = ['repo' => $this->getLockedRepository(true), 'method' => 'getDevRequires', 'description' => 'Required (in require-dev)'];
} }
$rootRepo = new RootPackageRepository($package);
foreach ($sets as $set) { foreach ($sets as $set) {
$installedRepo = new InstalledRepository([$set['repo']]); $installedRepo = new InstalledRepository([$set['repo'], $rootRepo]);
foreach (call_user_func([$package, $set['method']]) as $link) { foreach (call_user_func([$package, $set['method']]) as $link) {
if (PlatformRepository::isPlatformPackage($link->getTarget())) { if (PlatformRepository::isPlatformPackage($link->getTarget())) {
@ -527,9 +529,21 @@ class Locker
} }
if ($installedRepo->findPackagesWithReplacersAndProviders($link->getTarget(), $link->getConstraint()) === []) { if ($installedRepo->findPackagesWithReplacersAndProviders($link->getTarget(), $link->getConstraint()) === []) {
$results = $installedRepo->findPackagesWithReplacersAndProviders($link->getTarget()); $results = $installedRepo->findPackagesWithReplacersAndProviders($link->getTarget());
if ($results !== []) { if ($results !== []) {
$provider = reset($results); $provider = reset($results);
$missingRequirementInfo[] = '- ' . $set['description'].' package "' . $link->getTarget() . '" is in the lock file as "'.$provider->getPrettyVersion().'" but that does not satisfy your constraint "'.$link->getPrettyConstraint().'".'; $description = $provider->getPrettyVersion();
if ($provider->getName() !== $link->getTarget()) {
foreach (['getReplaces' => 'replaced as %s by %s', 'getProvides' => 'provided as %s by %s'] as $method => $text) {
foreach (call_user_func([$provider, $method]) as $providerLink) {
if ($providerLink->getTarget() === $link->getTarget()) {
$description = sprintf($text, $providerLink->getPrettyConstraint(), $provider->getPrettyName().' '.$provider->getPrettyVersion());
break 2;
}
}
}
}
$missingRequirementInfo[] = '- ' . $set['description'].' package "' . $link->getTarget() . '" is in the lock file as "'.$description.'" but that does not satisfy your constraint "'.$link->getPrettyConstraint().'".';
} else { } else {
$missingRequirementInfo[] = '- ' . $set['description'].' package "' . $link->getTarget() . '" is not present in the lock file.'; $missingRequirementInfo[] = '- ' . $set['description'].' package "' . $link->getTarget() . '" is not present in the lock file.';
} }

View File

@ -17,7 +17,15 @@ The locked version will not get overwritten by an install but fails on invalid p
"require": { "require": {
"foo/bar": "2.0.0", "foo/bar": "2.0.0",
"foo/baz": "2.0.0", "foo/baz": "2.0.0",
"foo/self": "self.version" "foo/self": "self.version",
"foo/provided": "1.0.0",
"foo/provided-wrong-version": "1.0.0",
"foo/root-replaced": "1.0.0",
"foo/root-replaced-wrong-version": "1.0.0"
},
"replace": {
"foo/root-replaced": "^1",
"foo/root-replaced-wrong-version": "^2"
} }
} }
--LOCK-- --LOCK--
@ -25,7 +33,14 @@ The locked version will not get overwritten by an install but fails on invalid p
"packages": [ "packages": [
{ "name": "foo/bar", "version": "1.0.0" }, { "name": "foo/bar", "version": "1.0.0" },
{ "name": "foo/baz", "version": "2.0.0" }, { "name": "foo/baz", "version": "2.0.0" },
{ "name": "foo/self", "version": "1.2.2" } {
"name": "foo/self",
"version": "1.2.2",
"provide": {
"foo/provided": "^1",
"foo/provided-wrong-version": "^3"
}
}
], ],
"packages-dev": [], "packages-dev": [],
"aliases": [], "aliases": [],
@ -45,6 +60,8 @@ install
Installing dependencies from lock file (including require-dev) Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform. Verifying lock file contents can be installed on current platform.
- Required package "foo/bar" is in the lock file as "1.0.0" but that does not satisfy your constraint "2.0.0". - Required package "foo/bar" is in the lock file as "1.0.0" but that does not satisfy your constraint "2.0.0".
- Required package "foo/provided-wrong-version" is in the lock file as "provided as ^3 by foo/self 1.2.2" but that does not satisfy your constraint "1.0.0".
- Required package "foo/root-replaced-wrong-version" is in the lock file as "replaced as ^2 by __root__ 1.2.3" but that does not satisfy your constraint "1.0.0".
This usually happens when composer files are incorrectly merged or the composer.json file is manually edited. This usually happens when composer files are incorrectly merged or the composer.json file is manually edited.
Read more about correctly resolving merge conflicts https://getcomposer.org/doc/articles/resolving-merge-conflicts.md Read more about correctly resolving merge conflicts https://getcomposer.org/doc/articles/resolving-merge-conflicts.md
and prefer using the "require" command over editing the composer.json file directly https://getcomposer.org/doc/03-cli.md#require-r and prefer using the "require" command over editing the composer.json file directly https://getcomposer.org/doc/03-cli.md#require-r