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

Add more IFLA_ values #2020

Merged
merged 1 commit into from Jan 5, 2021
Merged

Add more IFLA_ values #2020

merged 1 commit into from Jan 5, 2021

Conversation

joshtriplett
Copy link
Member

No description provided.

@rust-highfive
Copy link

r? @JohnTitor

(rust-highfive has picked a reviewer for you, use r? to override)

@joshtriplett
Copy link
Member Author

I'm expecting CI to fail here, due to values not available in headers. Waiting to see the failures, to determine how to address them.

@joshtriplett
Copy link
Member Author

OK, this now passes CI, and I'd like to merge it.

Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@JohnTitor
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Jan 5, 2021

📌 Commit 436948c has been approved by JohnTitor

@bors
Copy link
Contributor

bors commented Jan 5, 2021

⌛ Testing commit 436948c with merge f275dd9...

bors added a commit that referenced this pull request Jan 5, 2021
@bors
Copy link
Contributor

bors commented Jan 5, 2021

💔 Test failed - checks-actions

@JohnTitor
Copy link
Member

Seems we should skip more consts on musl.

@joshtriplett
Copy link
Member Author

Looks that way, sigh. I really wish musl would just use the real kernel headers.

@joshtriplett
Copy link
Member Author

@bors r=JohnTitor

@bors
Copy link
Contributor

bors commented Jan 5, 2021

📌 Commit 2076d1c has been approved by JohnTitor

@bors
Copy link
Contributor

bors commented Jan 5, 2021

⌛ Testing commit 2076d1c with merge 6deb8a9...

@bors
Copy link
Contributor

bors commented Jan 5, 2021

☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing 6deb8a9 to master...

@bors bors merged commit 6deb8a9 into rust-lang:master Jan 5, 2021
@joshtriplett joshtriplett deleted the ifla branch January 8, 2021 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants