tolumide-ng requested wasmtime-core-reviewers for a review on PR #12805.
tolumide-ng requested cfallin for a review on PR #12805.
tolumide-ng opened PR #12805 from tolumide-ng:fix/wasmtime-c-api-gc-update to bytecodealliance:main:
The
c-apicrate currently enables thegcfeature on itswasmtimedependency, even though it exposes agcfeature to control this behaviour. This prevents downstream users from disablinggc, sinceCargo.tomldoesn't allow overriding it. However, structs likeRootScopeare only available whengcis enabled. Removing it from default features, without other options, would break internal implementations.Changes
- Enable the
gcfeature via the wasmtime-c-api crate by default.- Removes the explicitly set
gcfeature for thewasmtimedependency inc-api, thus enabling downstream users to specify this behaviour.Closes https://github.com/bytecodealliance/wasmtime/issues/12783
github-actions[bot] added the label wasmtime:c-api on PR #12805.
alexcrichton submitted PR review.
alexcrichton created PR review comment:
Oh for this the
defaultfeature lives here instead which includesgc, so I think it's reasonable to just from[dependencies]
tolumide-ng requested wasmtime-default-reviewers for a review on PR #12805.
tolumide-ng updated PR #12805.
tolumide-ng updated PR #12805.
tolumide-ng created PR review comment:
Thank you for your response @alexcrichton, I am missing part of your suggestion here. Do you mean removing
gcas a default feature in' wasmtime/Cargo. toml' and adding#[cfg(feature = "gc")]to the affected parts?
tolumide-ng submitted PR review.
tolumide-ng updated PR #12805.
tolumide-ng requested alexcrichton for a review on PR #12805.
Last updated: Mar 23 2026 at 16:19 UTC