Stream: git-wasmtime

Topic: wasmtime / PR #8463 Start migrating some Wasmtime crates ...


view this post on Zulip Wasmtime GitHub notifications bot (Apr 24 2024 at 19:59):

alexcrichton requested cfallin for a review on PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 24 2024 at 19:59):

alexcrichton opened PR #8463 from alexcrichton:initial-no-std to bytecodealliance:main:

This commit is the first in what will be multiple PRs to migrate Wasmtime to being compatible with #![no_std]. This work is outlined in #8341 and the rough plan I have in mind is to go on a crate-by-crate basis and use CI as a "ratchet" to ensure that no_std compat is preserved. In that sense this PR is a bit of a template for future PRs.

This PR migrates a few small crates to no_std, basically those that need no changes beyond simply adding the attribute. The nontrivial parts introduced in this PR are:

The first point should provide CI checks that compatibility with no_std indeed works, at least from an "it compiles" perspective. Note that it's not sufficient to test with a target like x86_64-unknown-linux-gnu because extern crate std will work on that target, even when #![no_std] is active.

The second point however is likely to increase maintenance burden in Wasmtime unfortunately. Namely we'll inevitably, either here or in the future, forget to turn on some feature for some crate that's not covered in CI checks. While I've tried to do my best here in covering it there's no guarantee that everything will work and the combinatorial explosion of what could be checked in CI can't all be added to CI. Instead we'll have to rely on bug fixes, users, and perhaps point releases to add more use cases to CI over time as we see fit.

<!--
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
-->

view this post on Zulip Wasmtime GitHub notifications bot (Apr 24 2024 at 19:59):

alexcrichton requested wasmtime-compiler-reviewers for a review on PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 24 2024 at 19:59):

alexcrichton requested wasmtime-core-reviewers for a review on PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 24 2024 at 19:59):

alexcrichton requested fitzgen for a review on PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 24 2024 at 19:59):

alexcrichton requested wasmtime-default-reviewers for a review on PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 24 2024 at 19:59):

alexcrichton commented on PR #8463:

I'll also note that, procedurally, my plan is to discuss https://github.com/bytecodealliance/wasmtime/issues/8341 in the Wasmtime meeting tomorrow and assuming no objections start landing PRs, but the goal here isn't to jump the gun by any means but rather just be prepared.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 24 2024 at 20:17):

alexcrichton updated PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 24 2024 at 20:18):

alexcrichton updated PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 25 2024 at 19:52):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 25 2024 at 19:59):

alexcrichton updated PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 25 2024 at 19:59):

alexcrichton has enabled auto merge for PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 25 2024 at 20:31):

alexcrichton updated PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 25 2024 at 22:08):

alexcrichton updated PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 25 2024 at 22:08):

alexcrichton has enabled auto merge for PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 25 2024 at 22:29):

alexcrichton updated PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 25 2024 at 22:29):

alexcrichton has enabled auto merge for PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 25 2024 at 22:53):

alexcrichton updated PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 25 2024 at 22:53):

alexcrichton has enabled auto merge for PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 25 2024 at 23:02):

alexcrichton updated PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 25 2024 at 23:04):

alexcrichton updated PR #8463.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 25 2024 at 23:52):

alexcrichton merged PR #8463.


Last updated: Dec 23 2024 at 13:07 UTC