-
Notifications
You must be signed in to change notification settings - Fork 821
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
Fix filesystem rights on WASI, add integration test for file permissions #3240
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might need to follow up on the failing fd_append
tests. I feel like they could be legitimate test failures, although I'm curious why it only shows on nightly
and not the test suites for older compilers.
@syrusakbary This should pass now. I have disabled the https://github.com/wasmerio/wasmer/actions/runs/3290950297/jobs/5424476629#step:18:415 I think this can be merged, but I've opened #3250 for tracking this "regression". |
On host_fs this will fail because the root dir might not be empty
bors r+ |
Build succeeded: |
Fixes #3125