Skip to content

Use secp256k1-kmp#1788

Merged
sstone merged 2 commits intoandroidfrom
android-use-secp256k1-kmp
May 7, 2021
Merged

Use secp256k1-kmp#1788
sstone merged 2 commits intoandroidfrom
android-use-secp256k1-kmp

Conversation

@sstone
Copy link
Member

@sstone sstone commented May 7, 2021

We get rid of our old JNI wrapper and switch to bitcoin-lib 0.18 which is based on secp256k1-kmp.
This will give us a consistent secp256k1 library on all our apps (since secp256k1-kmp also provides an Android library).
We also switch from spongycastle bouncycastle (bitcoin-lib 0.18 uses bouncycastle).

@sstone sstone requested a review from dpad85 May 7, 2021 14:51
We get rid of our old JNI wrapper and switch to bitcoin-lib 0.18 which is based on secp256k1-kmp.
This will give us a consistent secp256k1 library on all our apps (since secp256k1-kmp also provides an Android library).
We also switch from spongycastle to bouncycastle (bitcoin-lib 0.18 uses bouncycastle).
@sstone sstone force-pushed the android-use-secp256k1-kmp branch from 8b67bcf to 60ca19a Compare May 7, 2021 14:52
Copy link
Member

@dpad85 dpad85 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@dpad85
Copy link
Member

dpad85 commented May 7, 2021

Tested successfully on device.

@sstone sstone merged commit 5dd5502 into android May 7, 2021
@sstone sstone deleted the android-use-secp256k1-kmp branch May 7, 2021 18:00
dpad85 added a commit to ACINQ/phoenix that referenced this pull request May 11, 2021
See ACINQ/eclair#1788
Also updates the secp256k1 library.

Fixes #52
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