Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to polkadot-v1.11.0 #274

Closed
wants to merge 2 commits into from
Closed

Conversation

4meta5
Copy link
Contributor

@4meta5 4meta5 commented Aug 6, 2024

WIP Closes #205

  • WIP generic template
  • evm template

references

@4meta5 4meta5 added the breaking This is a breaking change label Aug 6, 2024
@4meta5 4meta5 self-assigned this Aug 6, 2024

This comment was marked as off-topic.

@4meta5
Copy link
Contributor Author

4meta5 commented Aug 6, 2024

Currently working through the following error:

error: failed to select a version for `futures-timer`.
    ... required by package `litep2p v0.6.2 (https://github.com/paritytech/litep2p?branch=master#3b664f7b)`
    ... which satisfies git dependency `litep2p` of package `sc-network v0.34.0 (https://github.com/paritytech/polkadot-sdk?tag=polkadot-v1.11.0#0bb62492)`
    ... which satisfies git dependency `sc-network` of package `parachain-template-node v1.0.0 (/Users/4meta5/oz/polkadot-runtime-template/generic-template/node)`
versions that meet the requirements `^3.0.3` are: 3.0.3

all possible versions conflict with previously selected packages.

  previously selected package `futures-timer v3.0.2`
    ... which satisfies dependency `futures-timer = "^3.0.1"` (locked to 3.0.2) of package `sc-basic-authorship v0.34.0 (https://github.com/paritytech/polkadot-sdk?tag=polkadot-v1.11.0#0bb62492)`
    ... which satisfies git dependency `sc-basic-authorship` of package `parachain-template-node v1.0.0 (/Users/4meta5/oz/polkadot-runtime-template/generic-template/node)`

failed to select a version for `futures-timer` which could resolve this conflict

Cargo update fails as well, which I would expect would be the command to fix the above error. It does not identify the substrate-runtime-fuzzer package.

cargo update                                                     ~/oz/polkadot-runtime-template/generic-template/node
    Updating crates.io index
    Updating git repository `https://github.com/paritytech/polkadot-sdk`
    Updating git repository `https://github.com/srlabs/substrate-runtime-fuzzer.git`
error: no matching package found
searched package name: `substrate-runtime-fuzzer`
perhaps you meant:      substrate-runtimes-fuzzers
location searched: https://github.com/srlabs/substrate-runtime-fuzzer.git
required by package `template-fuzzer v0.1.0 (/Users/4meta5/oz/polkadot-runtime-template/generic-template/template-fuzzer)`

@4meta5 4meta5 mentioned this pull request Aug 7, 2024
2 tasks
@4meta5
Copy link
Contributor Author

4meta5 commented Aug 10, 2024

Currently working through yanking subdepdenencies. Basically trying to edit the Cargo.lock until we get compilation. Would appreciate another set of eyes from anyone available to help @ozgunozerk @KitHat

I have started to follow the approach in https://stackoverflow.com/questions/60576277/yanked-subdependency-error-e0433-failed-to-resolve-could-not-find-rt-i cc in the branch amar-yanking-cargo-to-upgrade-dependencies but I am still hitting errors. It is because the checksum is wrong when I manually edit the Cargo.lock. This would be easier if cargo update worked but it does not because of the error posted in the comment above.

@ozgunozerk
Copy link
Collaborator

ozgunozerk commented Aug 10, 2024

I'll take a look at this on Monday. @4meta5 you can leave this issue to me if you are feeling unproductive spending time on it

@4meta5
Copy link
Contributor Author

4meta5 commented Aug 10, 2024

Thank you @ozgunozerk , I will take you up on that offer and reassign the issue to you :)

I'll close this PR, because it is essentially just find and replace for the version. There was no progress beyond that.

@4meta5 4meta5 closed this Aug 10, 2024
@4meta5 4meta5 deleted the amar-upgrade-to-polkadot1-11 branch August 12, 2024 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking This is a breaking change
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

🎁 [Feature Request]: upgrade polkadot-sdk to v1.13.0
2 participants