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

Make wasmtime the default when the feature is enabled #8855

Merged
2 commits merged into from
May 19, 2021

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented May 19, 2021

No description provided.

@bkchr bkchr added A0-please_review Pull request needs code review. B5-clientnoteworthy C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels May 19, 2021
@bkchr bkchr requested a review from kianenigma May 19, 2021 07:13
@bkchr bkchr requested a review from coriolinus May 19, 2021 07:14
Copy link
Contributor

@coriolinus coriolinus left a comment

Choose a reason for hiding this comment

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

Much simpler than #8847; I like it!

@bkchr
Copy link
Member Author

bkchr commented May 19, 2021

bot merge

@ghost
Copy link

ghost commented May 19, 2021

Waiting for commit status.

@ghost ghost merged commit 10cd9ce into master May 19, 2021
@ghost ghost deleted the bkchr-wasmtime-default branch May 19, 2021 08:29
@@ -51,7 +51,7 @@ pub struct TryRuntimeCmd {
#[structopt(
long = "wasm-execution",
value_name = "METHOD",
possible_values = &WasmExecutionMethod::enabled_variants(),
possible_values = &WasmExecutionMethod::variants(),
case_insensitive = true,
default_value = "Interpreted"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it intended that the default value is still interpreted here while in import-params it depends on the compilation flags?

nazar-pc pushed a commit to autonomys/substrate that referenced this pull request Aug 8, 2021
* Make wasmtime the default when the feature is enabled

* Update client/cli/src/arg_enums.rs

Co-authored-by: Kian Paimani <[email protected]>

Co-authored-by: Kian Paimani <[email protected]>
This pull request was closed.
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. C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants