Skip to content
This repository was archived by the owner on Jul 24, 2023. It is now read-only.

Fix building issues with rust 1.6 #24

Closed
wants to merge 3 commits into from

Conversation

babariviere
Copy link

Fix building issues with rust 1.6.

@MagaTailor
Copy link

The saboteur tries this crap again! At least he found out how to fix his "fix" this time.

@babariviere
Copy link
Author

Yup! But it doesn't build because of the rand package that's weird

@MagaTailor
Copy link

That's completely unrelated and due to changes in nightly rust 1.7.

@babariviere
Copy link
Author

Oh ok so how to fix this?

@MagaTailor
Copy link

You should open an issue with rand if they don't already know about this.

@babariviere
Copy link
Author

Ok but is my fix "good"? Because I need some advice

@MagaTailor
Copy link

It should work - did you test it by running the binary?

@babariviere
Copy link
Author

Yes it works

@MagaTailor
Copy link

Ok then, it's probably safe to merge.

@babariviere
Copy link
Author

Ok thanks

@MagaTailor
Copy link

Oh, and you don't have to report it anywhere - it's a known regression in nightly just as I'd suspected.

rust-lang/rust#30713

EDIT I can confirm the patched source builds (and works) correctly on 1.7 nightly from a few days ago.

@babariviere
Copy link
Author

oh ok i didn't notice that

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants