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

feat(secp256k1): partially ported secp256k1 #65

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

kewde
Copy link

@kewde kewde commented Feb 17, 2023

  • Adds secp256k1 to native crypto
  • Bumped secp256k1 from 2017 to newer version of 2022
  • Boost BJAM rebuild only happens on MacOS
  • Boost B2 build on Linux
  • Adds GitHub CI that builds master & PRs on MacOS & Ubuntu

I haven't tried to actually test this at all but I can already list a bunch of issues:

  • Figure out why MacOS build hangs on libsecp256k1.build-ios-i386_iPhoneSimulator
  • Remove unnecessary secp256k1 JNI files (marked in comments)
  • JavaScript side is currently broken (no base16 import)
  • I removed an optimization patch for secp256k1 from the original repo, this hard patched the window size for ecmult precomputation of 11 (default is 15) which can now be set through --with-ecmult-window. see original patch.
  • --with-asm=arm might be worth testing
  • --enable-module-schnorrsig might be worth exploring to provide schnorr signing as well

@kewde kewde marked this pull request as ready for review February 18, 2023 12:36
@@ -4,6 +4,8 @@

cmake_minimum_required(VERSION 3.4.1)

include_directories(${CMAKE_SOURCE_DIR}/../../../../android/jni/include)
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: probably not needed

@kewde kewde force-pushed the kewde/port-secp256k1 branch 2 times, most recently from bca36be to c2633ef Compare February 18, 2023 17:56
@@ -0,0 +1,43 @@
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: probably not needed

@@ -0,0 +1,614 @@
#ifndef _SECP256K1_
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: probably not needed

@kewde kewde force-pushed the kewde/port-secp256k1 branch from c2633ef to 13fc43d Compare February 18, 2023 18:22
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.

1 participant