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

fixed wasi support (I think) #97

Closed
wants to merge 3 commits into from
Closed

fixed wasi support (I think) #97

wants to merge 3 commits into from

Conversation

sunny-g
Copy link

@sunny-g sunny-g commented Jan 3, 2020

New to wasi, but I think these changes fix the build errors described in #92.

I likely missed/mis-implemented something, and I committed some cargo fmt changes + deny warnings as well, I can cherry-pick them out and make a new PR if necessary. The relevant changes are:

  • adding wasi as a dependency
  • replacing __wasi_fd_t with wasi::Fd in sys::wasi::{mod,impl}.rs
  • updating send and recv to use wasi::sock_send and wasi::sock_recv in ext.rs, lines 1293 and 1346.

Feel free to let me know what to fix.

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.

1 participant