-
Notifications
You must be signed in to change notification settings - Fork 231
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
v0.5.1 does not build on musl with --features all #405
Comments
Seems like these constant definitions are missing from libc for musl or musl itself. We can workaround this by disabling this API for musl, but ideally these constants would be added to libc. |
from what i can tell these constants actually come from the kernel headers ( |
This should be fixed with a new release of libc which includes rust-lang/libc#3136. |
same error for me, it's still open? |
@playmyswift yes. I don't think libc had a release which include the pr that fixed this issues. Once a new release is out I'll close this issue. Might be worth update our libc version and adding a CI check for musl. |
if you want a momentary workaround, put the following: [patch.crates-io]
libc = { git = 'https://github.com/rust-lang/libc.git', rev = "e71a4c0b1fd40050f50844e2a752576bcb6b5edb" } into Cargo.toml of your project that uses this, and run of course, this has no warranty, and is /generally/ not great practice. use it as a workaround only :) 0.2.140 was just released, so the next one is anywhere from a few weeks to months away, who knows. 0.2.141 will be the 'fixed' version. |
If someone opens a pr that bumps the version it could be hours away. It's very much a community driven project and it's usually very willing to make new releases. |
sure, that makes sense :) i just tend to not force people to make releases and wait, myself |
Bump version to 0.2.141 Bump version as requested in rust-lang/socket2#405
New release of libc was just merged rust-lang/libc#3177, so in a moment this will be fixed (after a manual update using |
This looks like it was broken in #359.
The text was updated successfully, but these errors were encountered: