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

Migrate dir to yanix #650

Closed
wants to merge 6 commits into from
Closed

Migrate dir to yanix #650

wants to merge 6 commits into from

Conversation

kubkon
Copy link
Member

@kubkon kubkon commented Nov 28, 2019

This commit migrates our internal libc::DIR and libc::dirent wrappers to the yanix crate.

NB this PR builds on #649.

Jakub Konka added 3 commits November 28, 2019 17:54
This commit adds `yanix` crate as a Unix dependency for `wasi-common`.
`yanix` stands for Yet Another Nix crate and is exactly what the name
suggests: a crate in the spirit of the `nix` crate featuring certain
adjusments and additions tailored specfically to its main application
area which are implementation of WASI syscalls.
Having introduced `yanix` crate as an in-house replacement for the
`nix` crate, this commit makes the necessary changes to `wasi-common`
to depend _only_ on `yanix` crate.
This commit migrates our internal `libc::DIR` and `libc::dirent`
wrappers to the `yanix` crate.
@kubkon kubkon force-pushed the migrate-dir-to-yanix branch from 7d5d9e9 to 69d0af8 Compare November 28, 2019 16:54
@kubkon kubkon added the wasi:impl Issues pertaining to WASI implementation in Wasmtime label Nov 28, 2019
@kubkon kubkon force-pushed the migrate-dir-to-yanix branch from 2260c91 to afe820d Compare November 29, 2019 09:54
@kubkon kubkon requested review from sunfishcode and peterhuene and removed request for sunfishcode November 29, 2019 12:17
@kubkon
Copy link
Member Author

kubkon commented Nov 29, 2019

cc @marmistrz

@kubkon
Copy link
Member Author

kubkon commented Dec 3, 2019

Closed in favour of #649 which includes those changes in a slightly different form.

@kubkon kubkon closed this Dec 3, 2019
@sunfishcode sunfishcode deleted the migrate-dir-to-yanix branch May 21, 2021 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wasi:impl Issues pertaining to WASI implementation in Wasmtime
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant