-
Notifications
You must be signed in to change notification settings - Fork 25
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
Make changes needed to build on musl #198
Conversation
I only checked that the project builds on "x86_64-unknown-linux-musl". This is pretty ugly, but maybe someone will want to use this as a starting point re: musl support. |
Neat! I'll think a bit about the best way to merge this, specifically Initial reaction: This PR serves to highlight the need to isolate libc functions in plib crate, and remove any need to touch individual utils with |
👍🏻. Some of the hacky special casing for musl will be unnecessary when rust-lang/libc#3190 is resolved, so one option would be to just wait on that. I made a PR to add some missing constants (rust-lang/libc#3908), but I would assume it's not easy/quick to get things merged into the |
2af5f08
to
94f3272
Compare
High level guidance thoughts:
|
7ca27ef
to
5340b68
Compare
5340b68
to
31a3ecf
Compare
On-going comments:
|
Closed in favor of #198 |
No description provided.