Stream: git-wasmtime

Topic: wasmtime / PR #7792 Make wasmtime dependency in wiggle an...


view this post on Zulip Wasmtime GitHub notifications bot (Jan 18 2024 at 09:03):

juntyr opened PR #7792 from juntyr:wasi-common-optional-wasmtime to bytecodealliance:main:

<!--
Please make sure you include the following information:

Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.html

Please ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->

I am trying to use wasi-common (and thus also wiggle) to provide a common WasiCtx implementation that is reused between wasmtime and a custom runtime (as briefly mentioned in https://github.com/bytecodealliance/wasmtime/issues/7776). At the moment, compiling these packages also compiles wasmtime, which is not supported on my platform. I propose to add an enabled-by-default feature to wasi-common and wiggle that guards all wasmtime-specific code and allows both packages to be compiled without a dependency on wasmtime using default-features = false.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 18 2024 at 09:03):

juntyr requested wasmtime-default-reviewers for a review on PR #7792.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 18 2024 at 09:03):

juntyr requested wasmtime-core-reviewers for a review on PR #7792.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 18 2024 at 09:03):

juntyr requested fitzgen for a review on PR #7792.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 18 2024 at 15:50):

alexcrichton submitted PR review:

Thanks!

view this post on Zulip Wasmtime GitHub notifications bot (Jan 18 2024 at 16:06):

juntyr updated PR #7792.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 18 2024 at 17:29):

alexcrichton merged PR #7792.


Last updated: Nov 22 2024 at 17:03 UTC