Skip to content

Conversation

@ascjones
Copy link
Contributor

Fixes #218

@ascjones
Copy link
Contributor Author

ascjones commented Jan 12, 2021

Fix for failure in multi-addr, possibly need to wait for release of https://crates.io/crates/sc-network which depends on the old libp2p? libp2p/rust-libp2p@aa2547e.

Also see paritytech/substrate#7877

@niklasad1
Copy link
Contributor

niklasad1 commented Jan 14, 2021

Fix for failure in multi-addr, possibly need to wait for release of https://crates.io/crates/sc-network which depends on the old libp2p

It is getting backported libp2p/rust-libp2p#1925

@mxinden
Copy link

mxinden commented Jan 14, 2021

Fix for failure in multi-addr, possibly need to wait for release of https://crates.io/crates/sc-network which depends on the old libp2p

It is getting backported libp2p/rust-libp2p#1925

Backport is released. Please update to parity-multiaddr v0.9.7.

@ascjones
Copy link
Contributor Author

ascjones commented Jan 14, 2021

Patched parity-multiaddr is a transient dependency so it now builds with no change. Thanks @mxinden 🚀

@ascjones ascjones requested a review from niklasad1 January 14, 2021 16:08
[dev-dependencies]
async-std = { version = "1.6.4", features = ["attributes"] }
env_logger = "0.7.1"
async-std = { version = "1.8.0", features = ["attributes"] }
Copy link
Contributor

@niklasad1 niklasad1 Jan 14, 2021

Choose a reason for hiding this comment

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

NOTE, this feature will be leaked into dependencies (it's fine but still.. the new feature resolver is stabilized and will fix this FWIW)

Copy link
Contributor

@niklasad1 niklasad1 left a comment

Choose a reason for hiding this comment

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

great, LGTM

@ascjones ascjones merged commit 6175cb7 into master Jan 14, 2021
@niklasad1 niklasad1 deleted the aj-update-substrate branch January 14, 2021 17:11
@ascjones ascjones mentioned this pull request Feb 5, 2021
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.

update substrate to 2.0.1

4 participants