alexcrichton opened PR #6390 from alexcrichton:wit-changes
to bytecodealliance:main
:
This PR integrates bytecodealliance/wasm-tools#1027 into Wasmtime. The main changes here are:
- WIT syntax is updated with WebAssembly/component-model#193
- Generated bindings in the
bindgen!
macro have been updated to reflect the new structure of WIT.- The accepted component model binary format has been updated to account for changes.
This PR disables wasi-http tests and the on-by-default feature because the WIT syntax has been updated but the submodule containing the WITs has not been updated yet so there's no way to get that building temporarily. Once that's updated then this can be reenabled.
<!--
Please make sure you include the following information:
If this work has been discussed elsewhere, please include a link to that
conversation. If it was discussed in an issue, just mention "issue #...".Explain why this change is needed. If the details are in an issue already,
this can be brief.Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.htmlPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
alexcrichton updated PR #6390.
alexcrichton updated PR #6390.
alexcrichton updated PR #6390.
alexcrichton updated PR #6390.
alexcrichton updated PR #6390.
alexcrichton updated PR #6390.
alexcrichton updated PR #6390.
alexcrichton has marked PR #6390 as ready for review.
alexcrichton requested elliottt for a review on PR #6390.
alexcrichton requested wasmtime-compiler-reviewers for a review on PR #6390.
alexcrichton requested wasmtime-core-reviewers for a review on PR #6390.
alexcrichton requested jameysharp for a review on PR #6390.
alexcrichton requested wasmtime-default-reviewers for a review on PR #6390.
alexcrichton requested pchickey for a review on PR #6390.
alexcrichton updated PR #6390.
pchickey submitted PR review.
sunfishcode submitted PR review.
sunfishcode submitted PR review.
sunfishcode created PR review comment:
What is this
main.wit
for?
alexcrichton created PR review comment:
Without any
*.wit
files the current implementation inwit-parser
returns an error during parsing saying "nopackage
header was found" since it doesn't have a special case for zero parsed files. I can follow-up though with a special case to handle that.
alexcrichton updated PR #6390.
alexcrichton updated PR #6390.
alexcrichton has enabled auto merge for PR #6390.
alexcrichton updated PR #6390.
alexcrichton has enabled auto merge for PR #6390.
alexcrichton updated PR #6390.
alexcrichton has enabled auto merge for PR #6390.
alexcrichton updated PR #6390.
alexcrichton merged PR #6390.
Last updated: Nov 22 2024 at 17:03 UTC