Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

sc-network: switch on default features for libp2p on non-wasm-builds #8088

Merged
1 commit merged into from
Feb 10, 2021

Conversation

gnunicorn
Copy link
Contributor

@gnunicorn gnunicorn commented Feb 9, 2021

currently sc-network doesn't build if separated from the rest of workspace (see libp2p/rust-libp2p#1952). This fix flips the dependency switch around: activates the libp2p default features and excludes some for wasm-builds.

@gnunicorn gnunicorn added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Feb 9, 2021
@gnunicorn gnunicorn requested review from tomaka and mxinden February 9, 2021 14:53
Copy link
Contributor

@tomaka tomaka left a comment

Choose a reason for hiding this comment

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

Well, fine by me if CI passes

@gnunicorn gnunicorn requested a review from romanb February 9, 2021 16:15
@gnunicorn
Copy link
Contributor Author

bot merge

@ghost
Copy link

ghost commented Feb 10, 2021

Trying merge.

@ghost ghost merged commit 28b950e into master Feb 10, 2021
@ghost ghost deleted the ben-unbreak-unleash branch February 10, 2021 08:42
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants