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

fixes to #6462: make wasi-common API compatible with 9.0.1 #6471

Merged
merged 2 commits into from
May 26, 2023

Conversation

pchickey
Copy link
Contributor

required in order to make 9.0.2 a valid patch release.

  • struct FdStat is public, so I backed out changes to it, and manually applied the rights bits in the implementation of fd_fdstat_get.

  • new FileAccessMode changed to be pub(crate), to make sure it doesnt appear in any pub interfaces. WasiCtx::insert_file and push_file reverted to their original type signature. Private helper insert_file_ used for the set_{stdio} implementations.

@pchickey pchickey requested a review from a team as a code owner May 26, 2023 17:30
@pchickey pchickey requested review from alexcrichton and removed request for a team May 26, 2023 17:30
@pchickey pchickey requested a review from a team as a code owner May 26, 2023 17:30
Pat Hickey added 2 commits May 26, 2023 10:36
required in order to make 9.0.2 a valid patch release.

* `struct FdStat` is public, so I backed out changes to it, and manually
applied the rights bits in the implementation of fd_fdstat_get.

* new `FileAccessMode` changed to be pub(crate), to make sure it doesnt
appear in any pub interfaces. WasiCtx::insert_file and push_file
reverted to their original type signature. Private helper insert_file_
used for the set_{stdio} implementations.
@pchickey pchickey force-pushed the pch/release-9-wasi-api-compatible branch from 0f5e179 to 22292e6 Compare May 26, 2023 17:36
@github-actions github-actions bot added wasi Issues pertaining to WASI wasmtime:docs Issues related to Wasmtime's documentation labels May 26, 2023
@github-actions
Copy link

Subscribe to Label Action

cc @kubkon

This issue or pull request has been labeled: "wasi", "wasmtime:docs"

Thus the following users have been cc'd because of the following labels:

  • kubkon: wasi

To subscribe or unsubscribe from this label, edit the .github/subscribe-to-label.json configuration file.

Learn more.

@pchickey pchickey merged commit 32afef6 into release-9.0.0 May 26, 2023
@pchickey pchickey deleted the pch/release-9-wasi-api-compatible branch May 26, 2023 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wasi Issues pertaining to WASI wasmtime:docs Issues related to Wasmtime's documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants