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

Remove rust-sodium dependency #1102

Merged
merged 1 commit into from Dec 5, 2019
Merged

Conversation

@lionel1704
Copy link
Member

lionel1704 commented Dec 5, 2019

  • use tiny_keecak for generate_account_secrets
  • remove unused signing utilities module
  • remove rust-sodium dependency
- use tiny_keecak for generate_account_secrets
- remove unused signing utilities module
- remove rust-sodium dependency
@lionel1704 lionel1704 requested a review from nbaksalyar as a code owner Dec 5, 2019
Copy link
Member

nbaksalyar left a comment

Great stuff! 🎉

@lionel1704 lionel1704 merged commit 49df10d into maidsafe:master Dec 5, 2019
8 of 10 checks passed
8 of 10 checks passed
Rustfmt-Clippy
Details
Build Scripts
Details
build-android (armv7-linux-androideabi, prod)
Details
build-android (armv7-linux-androideabi, dev)
Details
build-android (x86_64-linux-android, prod)
Details
build-android (x86_64-linux-android, dev)
Details
Test (ubuntu-latest)
Details
Test (windows-latest)
Details
Travis CI - Pull Request Build Failed
Details
continuous-integration/jenkins/pr-merge This commit is being built
Details
@lionel1704 lionel1704 deleted the lionel1704:remove-rust-sodium branch Dec 5, 2019
@kpp

This comment has been minimized.

Copy link

kpp commented Dec 8, 2019

What do you use instead of libsodium?

@m-cat

This comment has been minimized.

Copy link
Contributor

m-cat commented Dec 8, 2019

@kpp

This comment has been minimized.

Copy link

kpp commented Dec 8, 2019

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.