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

Switch to wasm-builder 1.0.7 #441

Merged
merged 2 commits into from
Sep 23, 2019
Merged

Switch to wasm-builder 1.0.7 #441

merged 2 commits into from
Sep 23, 2019

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented Sep 23, 2019

No description provided.

"-C link-arg=--import-memory",
);

if env::var("TARGET") == Ok("wasm32-unknown-unknown".into()) {
println!("cargo:rustc-cfg=no_std");
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure I understand why this is needed. didn't you remove the no_std feature ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Ahhh :( Some leftovers.

@gui1117
Copy link
Contributor

gui1117 commented Sep 23, 2019

also you need to remove the core feature of adder in parachain/Cargo.toml

@bkchr
Copy link
Member Author

bkchr commented Sep 23, 2019

also you need to remove the core feature of adder in parachain/Cargo.toml

Which core feature?

@gui1117
Copy link
Contributor

gui1117 commented Sep 23, 2019

also you need to remove the core feature of adder in parachain/Cargo.toml

Which core feature?

None my bad it was on my local branch

@gui1117 gui1117 merged commit 6659d9a into master Sep 23, 2019
@gui1117 gui1117 deleted the bkchr-wasm-builder-1.0.7 branch September 23, 2019 10:38
@gui1117
Copy link
Contributor

gui1117 commented Sep 23, 2019

(should we backport stuff ? I don't how is strategy in this regard now)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants