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

Add new features to WasiState builder API #1170

Merged
merged 4 commits into from
Jan 22, 2020

Conversation

MarkMcCaskey
Copy link
Contributor

@MarkMcCaskey MarkMcCaskey commented Jan 22, 2020

Allows for ergonomic swapping out of stdin, stdout, stderr as well as
a new PreopenDir sub-builder which enables for user control over the
permissions that the directory is preopened with.

This PR includes breaking changes to the pubilc API.

TODO:

  • Add doc examples of using the new sub-builder
  • Discuss and maybe readd removed broken API with deprecated attributes on them

Review

  • Add a short description of the the change to the CHANGELOG.md file

Allows for ergonomic swapping out of stdin, stdout, stderr as well as
a new PreopenDir sub-builder which enables for user control over the
permissions that the directory is preopened with.

This PR includes breaking changes to the pubilc API.
@MarkMcCaskey MarkMcCaskey added the 📦 lib-wasi About wasmer-wasi label Jan 22, 2020
lib/wasi/src/state/builder.rs Outdated Show resolved Hide resolved
lib/wasi/src/state/builder.rs Outdated Show resolved Hide resolved
@MarkMcCaskey
Copy link
Contributor Author

bors r+

bors bot added a commit that referenced this pull request Jan 22, 2020
1170: Add new features to WasiState builder API r=MarkMcCaskey a=MarkMcCaskey

Allows for ergonomic swapping out of stdin, stdout, stderr as well as
a new PreopenDir sub-builder which enables for user control over the
permissions that the directory is preopened with.

This PR includes breaking changes to the pubilc API.

TODO:
- [x] Add doc examples of using the new sub-builder
- [x] Discuss and maybe readd removed broken API with `deprecated` attributes on them

# Review

- [x] Add a short description of the the change to the CHANGELOG.md file


Co-authored-by: Mark McCaskey <mark@wasmer.io>
Co-authored-by: Mark McCaskey <5770194+MarkMcCaskey@users.noreply.github.com>
@bors
Copy link
Contributor

bors bot commented Jan 22, 2020

Build succeeded

@bors bors bot merged commit 4862828 into master Jan 22, 2020
@bors bors bot deleted the feature/improved-wasi-fs-builder-api branch January 22, 2020 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📦 lib-wasi About wasmer-wasi
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants