-
Notifications
You must be signed in to change notification settings - Fork 674
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
Raise minimum supported Rust version to 1.36.0 #1108
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR. Could you please note in the commit message why you're raising MSRV (for mem::MaybeUninit). Also, don't forget to update the MSRV in README.md.
The minimum supported Rust version is being raised to 1.36.0 as this is the first release to support the `mem::MaybeUninit` feature. Signed-off-by: Otavio Salvador <[email protected]>
@asomers I rebased this on top of current master; please take a look. |
Please note in the commit message the reason for raising the msrv. Otherwise, it looks good. |
It is, on the long description. |
Ahh, my apologies. I didn't see it on my tiny cell phone screen. I'll take a look at the musl failure. |
Weirdly, musl uses i32 to store the ioctl opcode
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bors r+
1108: Raise minimum supported Rust version to 1.36.0 r=asomers a=otavio Signed-off-by: Otavio Salvador <[email protected]> Co-authored-by: Otavio Salvador <[email protected]> Co-authored-by: Alan Somers <[email protected]>
Build succeeded
|
1126: Relax some dependency versions r=asomers a=atouchet The minimum supported Rust version was bumped in #1108 so these restrictions are no longer necessary. Co-authored-by: Alex Touchet <[email protected]>
Signed-off-by: Otavio Salvador [email protected]