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

Bumping version of wasi crate dependency #3476

Merged
merged 6 commits into from
Mar 12, 2024

Conversation

landonxjames
Copy link
Contributor

@landonxjames landonxjames commented Mar 11, 2024

Motivation and Context

The 0.12.0 version of wasi suffered from a bug the could impact users using lower versions of wit-bindgen dependencies. Mixed versions of wit-bindgen generated bindings in a single project would cause a symbol collision. This is fixed by newer versions of wit-bindgen and wasi = 0.12.1 contains bindings generated by one of these newer versions. See this issue for details: bytecodealliance/wit-bindgen#849 and a longer discussion of the issue on Zulip here.

Description

Bumped version of wasi dependency in aws-amithy-wasm from 0.12.0 -> 0.12.1

Testing

Checklist

  • I have updated CHANGELOG.next.toml if I made changes to the smithy-rs codegen or runtime crates

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@landonxjames landonxjames requested a review from a team as a code owner March 11, 2024 21:41
0.12.0 suffered from a bug the could impact users using lower versions
of wit-bindgen dependencies. Mixed versions of wit-bindgen generated
bindings in a single project would cause a symbol collision. This is
fixed by newer versions of wit-bindgen and wasi 0.12.1 contains bindings
generated by one of these newer versions. See issue below for details:
bytecodealliance/wit-bindgen#849

Update CHANGELOG
CHANGELOG.next.toml Outdated Show resolved Hide resolved
CHANGELOG.next.toml Outdated Show resolved Hide resolved
@jdisanti jdisanti enabled auto-merge March 12, 2024 22:39
@jdisanti jdisanti added this pull request to the merge queue Mar 12, 2024
Merged via the queue into smithy-lang:main with commit 1e55d75 Mar 12, 2024
39 checks passed
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