Stream: git-wasmtime

Topic: wasmtime / PR #11687 Don't break cwasm ABIs in patch rele...


view this post on Zulip Wasmtime GitHub notifications bot (Sep 11 2025 at 19:01):

alexcrichton opened PR #11687 from alexcrichton:patches-dont-break-cwasm to bytecodealliance:main:

This commit is an update to our cwasm-version-matching logic to take #11685 into account. To me it feels reasonable to expect that patch releases of Wasmtime by default do not affect the ABI of *.cwasm artifacts and thus we should allow by default loading previous versions. This can help embedders avoid recompiles and can additionally assist in more quickly deploying security updates as recompilation is unnecessary.

The concrete change in this commit is that instead of baking in CARGO_PKG_VERSION into the cwasm artifact instead CARGO_PKG_VERSION_MAJOR is baked in. This means that the patch release is "forgotten" and won't be compared.

This commit also updates documentation for patch releases to indicate that this should be double-checked on patch releases. Furthermore we can always update the string ourselves manually (e.g. to include a ".2" at the end or something like that) if an ABI break is actually needed. This mostly just changes the default of patch releases by default don't break ABI but we still can if we really need to for a particular issue.

<!--
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 (Sep 11 2025 at 19:01):

alexcrichton requested dicej for a review on PR #11687.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 11 2025 at 19:01):

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

view this post on Zulip Wasmtime GitHub notifications bot (Sep 11 2025 at 19:01):

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

view this post on Zulip Wasmtime GitHub notifications bot (Sep 11 2025 at 20:38):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 11 2025 at 21:32):

dicej submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 11 2025 at 22:13):

alexcrichton merged PR #11687.


Last updated: Dec 06 2025 at 06:05 UTC