Просмотр исходного кода

Attempt to prune versions which are impossible to install during pool building

Nils Adermann 6 лет назад
Родитель
Сommit
83efeaec5c

+ 4 - 4
src/Composer/DependencyResolver/Pool.php

@@ -58,12 +58,12 @@ class Pool implements \Countable
 
     public function setPackages(array $packages, array $priorities = array())
     {
-        $this->priorities = $priorities;
-        $this->packages = $packages;
-
         $id = 1;
 
-        foreach ($this->packages as $package) {
+        foreach ($packages as $i => $package) {
+            $this->packages[] = $package;
+            $this->priorities[] = isset($priorities[$i]) ? $priorities[$i] : 0;
+
             $package->id = $id++;
             $names = $package->getNames();
             $this->packageByExactName[$package->getName()][$package->id] = $package;

+ 37 - 2
src/Composer/DependencyResolver/PoolBuilder.php

@@ -20,6 +20,8 @@ use Composer\Repository\ComposerRepository;
 use Composer\Repository\InstalledRepositoryInterface;
 use Composer\Repository\LockArrayRepository;
 use Composer\Repository\PlatformRepository;
+use Composer\Semver\Constraint\Constraint;
+use Composer\Semver\Constraint\MultiConstraint;
 
 /**
  * @author Nils Adermann <naderman@naderman.de>
@@ -30,6 +32,9 @@ class PoolBuilder
     private $filterRequires;
     private $rootAliases;
 
+    private $aliasMap = array();
+    private $nameConstraints = array();
+
     private $loadedNames = array();
 
     private $packages = array();
@@ -52,6 +57,7 @@ class PoolBuilder
             switch ($job['cmd']) {
                 case 'install':
                     $loadNames[$job['packageName']] = $job['constraint'];
+                    $this->nameConstraints[$job['packageName']] = $job['constraint'] ? new MultiConstraint(array($job['constraint']), false) : null;
                     break;
             }
         }
@@ -92,6 +98,17 @@ class PoolBuilder
             $loadNames = $newLoadNames;
         }
 
+        foreach ($this->packages as $i => $package) {
+            if (!$package instanceof AliasPackage && !isset($this->aliasMap[spl_object_hash($package)]) && isset($this->nameConstraints[$package->getName()])) {
+                $constraint = $this->nameConstraints[$package->getName()];
+
+                if ($constraint && !$constraint->matches(new Constraint('==', $package->getVersion()))) {
+                    unset($this->packages[$i]);
+                    unset($this->priorities[$i]);
+                }
+            }
+        }
+
         foreach ($repositories as $key => $repository) {
             if ($repository instanceof PlatformRepository ||
                 $repository instanceof InstalledRepositoryInterface) {
@@ -111,18 +128,26 @@ class PoolBuilder
         $this->packages[] = $package;
         $this->priorities[] = -$repoIndex;
 
+        if ($package instanceof AliasPackage) {
+            $this->aliasMap[spl_object_hash($package->getAliasOf())][] = $package;
+        }
+
         // handle root package aliases
         $name = $package->getName();
         if (isset($this->rootAliases[$name][$package->getVersion()])) {
             $alias = $this->rootAliases[$name][$package->getVersion()];
             if ($package instanceof AliasPackage) {
-                $package = $package->getAliasOf();
+                $basePackage = $package->getAliasOf();
+            } else {
+                $basePackage = $package;
             }
-            $aliasPackage = new AliasPackage($package, $alias['alias_normalized'], $alias['alias']);
+            $aliasPackage = new AliasPackage($basePackage, $alias['alias_normalized'], $alias['alias']);
             $aliasPackage->setRootPackageAlias(true);
 
             $package->getRepository()->addPackage($aliasPackage); // TODO do we need this?
             $this->packages[] = $aliasPackage;
+            $this->priorities[] = -$repoIndex;
+            $this->aliasMap[spl_object_hash($aliasPackage->getAliasOf())][] = $aliasPackage;
         }
 
         $loadNames = array();
@@ -131,6 +156,16 @@ class PoolBuilder
             if (!isset($this->loadedNames[$require])) {
                 $loadNames[$require] = null;
             }
+            if ($link->getConstraint()) {
+                if (!array_key_exists($require, $this->nameConstraints)) {
+                    $this->nameConstraints[$require] = new MultiConstraint(array($link->getConstraint()), false);
+                } elseif ($this->nameConstraints[$require]) {
+                    // TODO addConstraint function?
+                    $this->nameConstraints[$require] = new MultiConstraint(array_merge(array($link->getConstraint()), $this->nameConstraints[$require]->getConstraints()), false);
+                }
+            } else {
+                $this->nameConstraints[$require] = null;
+            }
         }
 
         return $loadNames;