Skip to content

Fix depencencies#996

Merged
claravanstaden merged 2 commits into
polkadot-fellows:pandres95-upgrade-to-stable2509from
encointer:cl/fix-dependencies
Nov 4, 2025
Merged

Fix depencencies#996
claravanstaden merged 2 commits into
polkadot-fellows:pandres95-upgrade-to-stable2509from
encointer:cl/fix-dependencies

Conversation

@clangenb

@clangenb clangenb commented Nov 4, 2025

Copy link
Copy Markdown
Contributor

This PR fixes the dependency mess that we still have in the update branch.

The issues with encointer's dependencies was that we only introduced patch bumps for the stable202509 upgrade to keep being aligned with the polkadot-sdk releases (we messed up the rhythm, as the unstable branch was a major release for us). The patch bump did not propagate to our innermost dependencies in the lock file, as it was deemed compatible with the ones explicitly defined in the root toml.

So this PR fixes (I believe all) of the duplicate substrate dependencies by running

  1. cargo update -p encointer-crates-at-wrong-version
  2. psvm --version polkadot-stable2509-1

@clangenb

clangenb commented Nov 4, 2025

Copy link
Copy Markdown
Contributor Author

cc @claravanstaden

@clangenb clangenb force-pushed the cl/fix-dependencies branch from 081b30e to af79daf Compare November 4, 2025 12:55
Comment thread Cargo.toml
Comment on lines 218 to +234
@@ -231,7 +231,7 @@ snowbridge-inbound-queue-primitives = { version = "0.5.0", default-features = fa
snowbridge-runtime-common = { version = "0.17.0", default-features = false }
snowbridge-runtime-test-common = { version = "0.19.0" }
snowbridge-system-runtime-api = { version = "0.16.0", default-features = false }
snowbridge-system-v2-runtime-api = { version = "0.4.0", default-features = false }
snowbridge-system-v2-runtime-api = { version = "0.5.0", default-features = false }

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@claravanstaden psvm --version polkadot-stable2509-1 yielded this. Can you confirm that this is ok?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is 100%, thank you!

@claravanstaden claravanstaden mentioned this pull request Nov 4, 2025
1 task
@claravanstaden claravanstaden merged commit 8796319 into polkadot-fellows:pandres95-upgrade-to-stable2509 Nov 4, 2025
16 of 71 checks passed
@clangenb clangenb deleted the cl/fix-dependencies branch December 3, 2025 06:02
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