Skip to content

wasmtime: Clarify that component::Linker doesn't support intra-component linking yet - #14088

Open
nightkr wants to merge 3 commits into
bytecodealliance:mainfrom
nightkr:docs/component-linker-limitations
Open

wasmtime: Clarify that component::Linker doesn't support intra-component linking yet#14088
nightkr wants to merge 3 commits into
bytecodealliance:mainfrom
nightkr:docs/component-linker-limitations

Conversation

@nightkr

@nightkr nightkr commented Aug 6, 2026

Copy link
Copy Markdown

Previously discussed at https://bytecodealliance.zulipchat.com/#narrow/channel/217126-wasmtime/topic/.E2.9C.94.20linking.20wasm.20components.20at.20runtime.3F/near/615012938

The current state tripped me up a bit, since the docs make it sound like it's already there and working, while the API itself seems nowhere to be found.

…ent linking yet

https://bytecodealliance.zulipchat.com/#narrow/channel/217126-wasmtime/topic/.E2.9C.94.20linking.20wasm.20components.20at.20runtime.3F/near/615012938

The current state tripped me up a bit, since the docs make it sound
like it's already there and working, while the API itself seems
nowhere to be found.
@nightkr
nightkr requested a review from a team as a code owner August 6, 2026 21:57
@nightkr
nightkr requested review from dicej and removed request for a team August 6, 2026 21:57

@dicej dicej left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@dicej
dicej enabled auto-merge August 6, 2026 22:10

@pchickey pchickey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of a footnote that it doesn't do what it says it does, can you just delete "both link components together as well as"? If component linking ever gets implemented, the docs can be updated then

@nightkr

nightkr commented Aug 6, 2026

Copy link
Copy Markdown
Author

@pchickey Sure, that also works for me. I didn't really want to disrupt the documented intent but ultimately it's you folks' call.

auto-merge was automatically disabled August 6, 2026 22:15

Head branch was pushed to by a user without write access

@nightkr
nightkr requested a review from pchickey August 6, 2026 22:16
@github-actions github-actions Bot added the wasmtime:api Related to the API of the `wasmtime` crate itself label Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wasmtime:api Related to the API of the `wasmtime` crate itself

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants