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

Ed25519 signing #871

Merged
merged 13 commits into from Oct 28, 2019

Conversation

@majestrate
Copy link
Collaborator

majestrate commented Oct 22, 2019

use normal ed25519 keys and signatures, remove screwy monero cryptography.

@majestrate majestrate marked this pull request as ready for review Oct 22, 2019
@majestrate

This comment has been minimized.

Copy link
Collaborator Author

majestrate commented Oct 22, 2019

this also fixes broken travis

majestrate added 7 commits Oct 22, 2019
@majestrate

This comment has been minimized.

Copy link
Collaborator Author

majestrate commented Oct 28, 2019

okay let's break the world :D

@majestrate majestrate merged commit 3a6c16a into loki-project:dev Oct 28, 2019
1 check was pending
1 check was pending
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
elseif(NOT ANDROID AND NOT NON_PC_TARGET)
if (NOT USE_AVX2)
# Public binary releases
set(CRYPTO_FLAGS -march=nocona -mtune=core2 -mfpmath=sse)
else()
set(CRYPTO_FLAGS -march=haswell -mtune=native -mfpmath=sse)
endif()
Comment on lines -141 to -147

This comment has been minimized.

Copy link
@despair86

despair86 Oct 29, 2019

Collaborator

this also affected NTRU

@neuroscr neuroscr referenced this pull request Nov 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.