Fixed dependency problem caused by pull request #3482#3493
Fixed dependency problem caused by pull request #3482#3493naderman merged 1 commit intocomposer:masterfrom
Conversation
|
Can you post your composer.json file that was experiencing this issue? |
|
Mind the PhantomInstaller commands at the bottom. They reference a bundle that's in our symfony installation--not one that's included in this file. |
|
Would be great if we can turn this into an installer functional test - see https://github.com/composer/composer/tree/master/tests/Composer/Test/Fixtures/installer Would prefer to fix this particular problem some other way if possible and not slow composer down by a fair amount again. |
|
I ran @naderman I don't believe this is a bug. |
|
With the current version of composer with my commit With the current version of composer without my commit |
|
@naderman it looks like the conflicts between the |
|
There are 3 other requires in a private repo which should have been included with the previous composer.json: |
|
@stof Indeed, I was pretty sure that was taken care of by the loop further up, but I guess I made a mistake there? |
|
@spreston what version of php are you running? |
|
5.5.9 |
|
As @stof said the replace stuff does not work as expected, here is a simple example: Using an old composer version it installs But when I use the latest composer it installs |
|
I have tried to create a failing test ...but it passes, so I think I'm doing something wrong here |
|
@naderman I think the issue is related to the optimization restricting packages based on the root package requirements. This can cause the opposite side of the conflict to not be added in the rule set. this issue has been reported again in #3507. |
|
Ah that's a good point, I was trying to come up with a decent test case for this, and ended up working on a tool to extract one from a composer.json ... But yeah I guess we might have to just slow down composer again and figure out a proper fix afterwards. |
Fixed dependency problem caused by pull request #3482
|
@naderman try looking at my idea about the root requirement optimization. Each report until now was about a package which was appearing in the root requirements too. |
|
Yeah, but I kind of want to have this test generation tool to debug other solve issues too now ;-) |
This was causing a Symfony 2.1 project to download dependencies from 2.7.