Skip to content

Fix build order transitive options - #20331

Draft
memsharded wants to merge 6 commits into
conan-io:develop2from
memsharded:fix-build-order-transitive-options
Draft

memsharded wants to merge 6 commits into
conan-io:develop2from
memsharded:fix-build-order-transitive-options

Conversation

@memsharded

Copy link
Copy Markdown
Member

Changelog: (Feature | Fix | Bugfix): Describe here your pull request
Docs: https://github.com/conan-io/docs/pull/XXXX

Counter examples for #20315 PR

@memsharded

memsharded commented Sep 11, 2026

Copy link
Copy Markdown
Member Author

Hi @AnastaZIuk

It seems this is way more challenging than what we initially thought. I have been finding more corner cases, adding more tests and trying to complete the implementation, but not happy about it yet, so this will require more time.

At the core of this issue lies the recommendation in https://docs.conan.io/2/knowledge/faq.html#defining-options-for-dependencies-in-conanfile-py-recipes-doesn-t-work:

So in general, defining options values for dependencies in recipes is discouraged. 
The strongly recommended way to define options values for dependencies is in profile files.

Defining options values in recipes introduce a data locality that makes the computation of partial parts of the dependency graph very challenging. This is why defining options values that deviate from the defaults in profiles is way simpler and more robust, I'd recommend doing that while we keep working on this, because this might take more time than expected. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants