Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multi version support #693

Merged
merged 4 commits into from
Oct 10, 2023
Merged

Conversation

rylev
Copy link
Contributor

@rylev rylev commented Oct 9, 2023

A continuation of bytecodealliance/wasmtime#7172 and bytecodealliance/wasmtime#7196 and borrows liberally from those PRs. This allows the use of different versions of the same logical interface.

This only fixes Rust so I'm marking it as a draft until tomorrow when I can fix the other languages.

rylev added 2 commits October 9, 2023 18:49
Signed-off-by: Ryan Levick <[email protected]>
Copy link
Member

@alexcrichton alexcrichton left a comment

Choose a reason for hiding this comment

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

Thanks!

@alexcrichton
Copy link
Member

I see that at least the C test is failing, and it's ok to add ignores to other generators for this new test if you'd like

@rylev rylev marked this pull request as ready for review October 10, 2023 08:21
@rylev
Copy link
Contributor Author

rylev commented Oct 10, 2023

@alexcrichton I added ignores to C, Go, and Java, and improved the test a bit to catch one additional issue I wasn't yet handling. This should be ready to merge. I can try to look into the other languages as well.

@alexcrichton alexcrichton merged commit 46fba30 into bytecodealliance:main Oct 10, 2023
@rylev rylev deleted the multi-version branch October 10, 2023 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants