[bug] Conanfile self.dependencies.items() is empty when overriding versions in the top package. about conan HOT 5 OPEN

Hi @devlpr

Thanks for your report.

I am not sure this is a bug, but intended and expected behavior. The override=True are not real dependencies, they are just a definition of a version change upstream, but it doesn't create a direct dependency from the caller to the required package.

I am not sure why this would be a problem, if you can please elaborate. This has worked always the same in Conan 1 and Conan 2, the behavior is exactly the same, so I don't fully understand why this would be a problem in the upgrade.

from conan.

We have not moved to Conan 2 yet. This is in Conan 1. The problem is that when we go to release, we run some operations on each package. When overriding versions, the list is empty, so we can't do those operations. So we can't use overrides. But sometimes we can't resolve versions, so we need overrides. It's a catch 22.

I think our expectation of an override is that the dependency tree is walked and when an overridden package is found, it simply uses the version in the override. It's overriding a dependency. So even if the override itself isn't a real dependency the actual dependencies should still act the same. By overriding, we aren't removing them. But that is how it's acting currently.

from conan.

Then if I understood correctly what you want is:

def requirements(self):
        self.requires("sdk/0.2.3@ai/release", force=True)
        self.requires("clock-synchronizer/1.0.4@ai/release", force=True)

This will work both in Conan 1 and Conan 2, overriding the upstream dependencies versions in both cases and being a direct dependency and listed in self.dependencies in both cases. Can you please give it a try and let me know?

An extra recommendation, just in case: overrides and forces are recommended only as temporary workarounds to upstream conflicts and version issues in the graph. The recommended and stable approach is to fix those issues in the recipes upstream

from conan.

Interesting. I will try this ASAP. I don't think we tried this yet. Thank you for this. If it works I owe you a beer!

As for overrides and forces, it's a long story how we got here, but I will say that we know better and we are trying to remedy our situation as soon as possible. With that fix, we are moving to Conan 2 so we can pick up the latest and greatest, and perhaps even contribute if needed.

from conan.

Interesting. I will try this ASAP. I don't think we tried this yet. Thank you for this. If it works I owe you a beer!

Always happy to help, just keep me posted, if this works that is the best for me too :)

As for overrides and forces, it's a long story how we got here, but I will say that we know better and we are trying to remedy our situation as soon as possible. With that fix, we are moving to Conan 2 so we can pick up the latest and greatest, and perhaps even contribute if needed.

One of the cool features in Conan 2 is the new [replace_requires] that also allows to temporarily fix conflicts without modifying conanfile.py recipes, so after fixing the upstream issues, it is not necessary to change the recipes again.

from conan.