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

Fix otp19 deprecation #177

Merged
merged 2 commits into from
Jun 24, 2016
Merged

Fix otp19 deprecation #177

merged 2 commits into from
Jun 24, 2016

Conversation

umbec
Copy link
Collaborator

@umbec umbec commented Jun 23, 2016

Since OTP 19 random has been deprecated in favour of rand.

Since OTP 19 crypto:rand_bytes/1 has been deprecated in favour of crypto:strong_rand_bytes/1.

See otp_src_19.0.readme.

Since OTP 19 crypto:rand_bytes/1 has been deprecated
in favour of crypto:strong_rand_bytes/1.
@etrepum
Copy link
Member

etrepum commented Jun 23, 2016

In order to remain compatible with older versions of Erlang this will need to be done conditionally with macros.

@umbec umbec force-pushed the fix_otp19_deprecation branch from 0f3775a to 8ddf8e7 Compare June 23, 2016 21:10
@umbec
Copy link
Collaborator Author

umbec commented Jun 23, 2016

More than reasonable!

I added the macros but the test still fails in some cases, i'll give it a look tomorrow.

@umbec umbec force-pushed the fix_otp19_deprecation branch 2 times, most recently from 90ce7d8 to 02cd4cb Compare June 24, 2016 09:05
Since OTP 19 random has been deprecated in favour of rand.
@umbec umbec force-pushed the fix_otp19_deprecation branch from 02cd4cb to 05dd6cd Compare June 24, 2016 10:16
@umbec
Copy link
Collaborator Author

umbec commented Jun 24, 2016

Did it.

@etrepum etrepum merged commit 5ab5017 into mochi:master Jun 24, 2016
@etrepum
Copy link
Member

etrepum commented Jun 24, 2016

Thanks! :)

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.

2 participants