Skip to content

Fix changing transitive statics package_id#19705

Merged
czoido merged 12 commits into
conan-io:develop2from
memsharded:fix/transitive_statics_package_id
Apr 21, 2026
Merged

Fix changing transitive statics package_id#19705
czoido merged 12 commits into
conan-io:develop2from
memsharded:fix/transitive_statics_package_id

Conversation

@memsharded

@memsharded memsharded commented Mar 3, 2026

Copy link
Copy Markdown
Member

Changelog: Bugfix: Change computation of package_id for transitive static libraries, based on the new required_conan_version=">=2.28" recipe version or global.conf using core:policies=["required_conan_version>=2.28"].
Docs: conan-io/docs#4438

For clarification, this bugfix cannot be introduced without the required_conan_version check because it would be breaking for many users, including ConanCenter that already have their binaries and their package_id depending on the previous behavior.

Close #19664

Comment thread conan/internal/model/requires.py Outdated
@memsharded memsharded added this to the 2.27.0 milestone Mar 4, 2026
@memsharded memsharded requested review from AbrilRBS and jcar87 March 5, 2026 16:35
@memsharded memsharded modified the milestones: 2.27.0, 2.28.0 Mar 23, 2026
@memsharded memsharded requested review from franramirez688 and removed request for AbrilRBS April 19, 2026 21:48
Comment thread conan/internal/graph/compute_pid.py
@memsharded

Copy link
Copy Markdown
Member Author

Did the review:

  • Using also global.conf global core:required_conan_version as input as well
  • Adding description message as example in the conan config list required_conan_version, so it lists the different policies applied

@memsharded memsharded marked this pull request as ready for review April 20, 2026 12:40
@memsharded memsharded requested a review from AbrilRBS April 20, 2026 12:40
Comment thread conan/internal/model/conf.py Outdated
Comment thread conan/internal/model/requires.py
@czoido czoido merged commit 1fafcf2 into conan-io:develop2 Apr 21, 2026
30 of 31 checks passed
@memsharded memsharded deleted the fix/transitive_statics_package_id branch April 21, 2026 08:47
memsharded added a commit to memsharded/conan that referenced this pull request Apr 22, 2026
* poc of changing transitive statics package_id

* review

* conf

* wip

* wip

* new approach with required_conan_version>=2.28

* fix

* wip

* remove help line
@memsharded memsharded changed the title poc of changing transitive statics package_id Fix changing transitive statics package_id Apr 28, 2026
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.

[bug] package ID /package info for static library contains indirect transitive dependencies

5 participants