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

Upgrade to rand 0.7.0 #18

Closed
wants to merge 3 commits into from
Closed

Conversation

fubuloubu
Copy link

fixes #17

@fubuloubu
Copy link
Author

Failure is due to support for v1.22

Unsure if this is a requirement, but seems like an older version that should be deprecated?

@newpavlov
Copy link
Member

Yes, rand v0.7 has MSRV equal to 1.32, so we we will have to bump minor version. I have some breaking-change API redesigns in mind, so we better bump rand together with this redesign.

@fubuloubu
Copy link
Author

Any chance we can bump out this small change separately of the breaking updates? It's currently blocking me on a project. Thanks!

@fubuloubu
Copy link
Author

Any updates on this?

@pdugre
Copy link

pdugre commented May 29, 2020

Any updates? It's blocking me too.

EDIT: Happened to me trying to compile scrypt in WASM, so this PR doesn not fix my problem.

@tarcieri
Copy link
Member

Superseded by #31

@tarcieri tarcieri closed this Jun 10, 2020
@fubuloubu fubuloubu deleted the patch-1 branch June 10, 2020 13:27
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.

Upgrade to rand 0.7.0
4 participants