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

Migrate rust-lang/rust to rand_core 0.7 #855

Closed
gnzlbg opened this issue Jul 31, 2019 · 5 comments
Closed

Migrate rust-lang/rust to rand_core 0.7 #855

gnzlbg opened this issue Jul 31, 2019 · 5 comments

Comments

@gnzlbg
Copy link

gnzlbg commented Jul 31, 2019

Every time a new release of rand happens, it would make sense to migrate rust-lang/rust to use it, since it is a huge user of rand that can discover oversights and design directions that might be problematic.

It is a particularly good place to test "future compatibility"-shims. These are hard to write, but the dependency graph of rustc is big enough to exercise them in many different ways.

@dhardy
Copy link
Member

dhardy commented Jul 31, 2019

I disagree about it being a good place to test compatibility shims — small examples are quite enough. The issue with rand 0.6.5 is simply that it violates semver rules as you pointed out — I don't believe this is worth fixing now, since doing so would require yanking crate versions already in widespread usage, downgrading to rand_core 0.3.x and would break no_std support for all Rand 0.6.x users (unless we instead pushed a breaking change to rand_core 0.3.x and hoped that caused less damage). Of course, we will try our best not to this again in the future 😉

Edit: of course, #852 is still unexplained.

As for upgrading rustc, I don't have the time myself this week and perhaps also not in the near future. Whoever gets there first I guess.

@gnzlbg
Copy link
Author

gnzlbg commented Jul 31, 2019

As for upgrading rustc, I don't have the time myself this week and perhaps also not in the near future. Whoever gets there first I guess.

I am just pointing out that probably nobody is going to try because it is currently already too much work, and this will mean that 0.6.1 and 0.7, newer 0.7.x versions, and in the future 0.8, 0.9, etc. will need to build and work properly in rust-lang/rust dependency graph. The more versions get released the more work it is to update, the harder it is going to be to find a workaround to fix things quickly if things break, etc.

@mati865
Copy link

mati865 commented Aug 2, 2019

I'll see what I can do about this issue.

@mati865
Copy link

mati865 commented Aug 5, 2019

Upgrading all Rust crates to rand 0.7 fail with the same error as upgrading to rand 0.6.5 #852

@dhardy
Copy link
Member

dhardy commented Aug 28, 2019

Thanks @mati865 for addressing this.

@dhardy dhardy closed this as completed Aug 28, 2019
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

No branches or pull requests

3 participants