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

WIP: Support all possible netlink constants in gnu/mod.rs on musl #1452

Closed
wants to merge 1 commit into from

Conversation

jbaublitz
Copy link
Contributor

This PR is meant to discover which netlink constants need to be gated in gnu/mod.rs and which ones are okay in mod.rs for Linux. It seems many of the early constants added were defaulted to GNU only which is not necessarily needed. This will probably fail CI a few times before I am able to remove WIP.

@rust-highfive
Copy link

r? @gnzlbg

(rust_highfive has picked a reviewer for you, use r? to override)

@jbaublitz jbaublitz force-pushed the musl-netlink-support branch from d51d3b7 to e4dcbbe Compare August 7, 2019 00:54
@gnzlbg gnzlbg closed this Aug 9, 2019
@gnzlbg gnzlbg reopened this Aug 9, 2019
@jbaublitz
Copy link
Contributor Author

I'm going to close this out and make a new PR. I set up my own musl-based CI that seems to work with this and the new pipelines that you mentioned in #1460 don't seem to be showing up.

@jbaublitz jbaublitz closed this Aug 10, 2019
bors added a commit that referenced this pull request Aug 10, 2019
Move musl-compatible constants to linux/mod.rs

This is a continuation of #1452. I think CI is ready to run fully and the PR is ready to be merged if the pipelines pass based on the CI I've done using musl libc in my own projects.
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.

None yet

3 participants