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

Compatibility with rust latest #94

Closed
wants to merge 1 commit into from

Conversation

zmack
Copy link

@zmack zmack commented Apr 4, 2015

Hey, I was trying to get some stuff done with mio on rust head, and just ended up in this sed-yak-shave, hope you can get some use out of it.

The only tricky bit is the clone implementation in UtsName, which i'm still not sure is 100% correct, but at least it gets tests passing.

Anyway, on this particular branch tests are green on today's rust. Hope it helps 😸

@carllerche
Copy link
Contributor

Yeah, I've been trying to get nix running on Rust beta. I already applied some of these fixes, but haven't yet pushed. I will attempt to get nix running on the latest nightly at least.

@fhartwig
Copy link
Contributor

fhartwig commented Apr 5, 2015

Re: the UtsName issue: apparently the recommended workaround is the one that I've used in #92

@zmack
Copy link
Author

zmack commented Apr 5, 2015

@fhartwig hah, nice, i was wondering about that one.

Good discussion on that as well down here, thanks for pointing me to it.

@carllerche
Copy link
Contributor

Thanks for the PR. Master should be fixed for Rust 1.0 Beta. I ended up merging #92

@carllerche carllerche closed this Apr 7, 2015
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.

3 participants