Migrate to upstream coinlib v5

.github/workflows/dart-test-analyze.yml

@@ -23,6 +23,10 @@ jobs:
         run: dart pub get
         working-directory: ./coinlib
 
+      - name: Verify formatting
+        run: dart format --output=none --set-exit-if-changed .
+        working-directory: ./coinlib
+
       - name: Analyze project source
         run: dart analyze --fatal-infos
         working-directory: ./coinlib

LICENSE

@@ -1,4 +1,4 @@
-Copyright 2023-2025 Peercoin Developers
+Copyright 2023-2026 Peercoin Developers
 
 Redistribution and use in source and binary forms, with or without modification,
 are permitted provided that the following conditions are met:

coinlib/CHANGELOG.md

@@ -1,3 +1,23 @@
+## 5.0.0
+
+Requires a rebuild of the secp256k1 library as it has moved to the
+secp256k1-coinlib fork.
+
+- Adds MuSig2 support with `MuSigPublicKeys` and `MuSigStatefulSigningSession`.
+- Includes adaptor signature support when using MuSig2 with
+  `SchnorrAdaptorSignature`.
+- Adds `InputSequence` abstraction for input sequences. Input sequences are now
+    `0xfffffffe` by default and enforce locktimes.
+- Adds `locktimeIsEnforced` to `Transaction` to determine if the locktime is in
+    effect.
+- Adds `Locktime` abstractions for transactions. Transactions have
+    `.isUnlocked` to determine if the transaction is available for block
+    inclusion.
+- Moves to underlying secp256k1-coinlib.
+- Removed dependency to wasm_interop that had a broken js dependency.
+- Fixes `extraEntropy` being ignored for `Secp256k1Base.schnorrSign`.
+- Adds MinGW build support for Windows
+
 ## 4.1.0
 
 `CoinSelection` now works for `TaprootKeyInput` and

coinlib/README.md

@@ -99,8 +99,9 @@ the root directory of your package which will produce a shared library into
 `build/libsecp256k1.dll`. This can also be run in the `coinlib` root directory
 via `dart run bin/build_windows.dart`.
 
-Windows builds use the Visual Studio 17 2022 generator.  Earlier Visual Studio
-toolchains may work by editing `bin/build_windows.dart`.
+Windows builds use the Visual Studio 17 2022 generator. Earlier Visual Studio
+toolchains may work by editing `bin/build_windows.dart`. MingGW can also be used
+with `dart run coinlib:build_windows_mingw`.
 
 ### Cross-compiling for Windows from Linux
 

coinlib/analysis_options.yaml

@@ -17,3 +17,6 @@ linter:
     - valid_regexps
     - require_trailing_commas
     - comment_references
+
+formatter:
+  trailing_commas: preserve

coinlib/bin/build_linux.dart

@@ -7,49 +7,45 @@ String dockerfile = r"""
 FROM debian:bookworm
 
 # Install dependenices
-RUN apt-get update -y \
-  && apt-get install -y autoconf libtool build-essential git
+RUN apt-get update -y && apt-get install -y cmake git
 
-# Clone libsecp256k1.
-# Could use secp256k1 already in code-base but this makes the dockerfile more
-# independent and avoids complexity of copying everything into the correct
-# context. It's not a large library to download.
-# Use 0.5.0 release
-RUN git clone https://github.com/bitcoin-core/secp256k1 \
-  && cd secp256k1 \
-  && git checkout e3a885d42a7800c1ccebad94ad1e2b82c4df5c65
+# Clone libsecp256k1-coinlib v0.7.0
+RUN git clone https://github.com/peercoin/secp256k1-coinlib \
+  && cd secp256k1-coinlib \
+  && git checkout 69018e5b939d8d540ca6b237945100f4ecb5681e
 
-WORKDIR /secp256k1
+WORKDIR /secp256k1-coinlib
 
 # Build shared library for linux
-RUN ./autogen.sh
-RUN ./configure \
-  --enable-module-recovery --disable-tests \
-  --disable-exhaustive-tests --disable-benchmark \
-  CFLAGS="-O2"
-RUN make
+RUN cmake -B build \
+      -DSECP256K1_ENABLE_MODULE_RECOVERY=ON \
+      -DSECP256K1_BUILD_TESTS=OFF \
+      -DSECP256K1_BUILD_EXHAUSTIVE_TESTS=OFF \
+      -DSECP256K1_BUILD_BENCHMARK=OFF \
+      -DSECP256K1_BUILD_EXAMPLES=OFF \
+      -DSECP256K1_BUILD_CTIME_TESTS=OFF \
+      -DCMAKE_BUILD_TYPE=Release
+RUN cmake --build build
 
 # Build shared library into /usr/local/lib as usual and then copy into output
 # Unused symbols could be stripped. But for future ease, all symbols are
 # maintained.
-RUN make install
+RUN cmake --install build
 RUN mkdir output
-RUN cp /usr/local/lib/libsecp256k1.so.2.2.0 output/libsecp256k1.so
+RUN cp /usr/local/lib/libsecp256k1.so.6.0.0 output/libsecp256k1.so
 """;
 
 void main() async {
-
   String cmd = await getDockerCmd();
   print("Using $cmd to run dockerfile");
 
   // Build secp256k1 and copy shared library to build directory
   if (!await dockerBuild(
-      cmd,
-      dockerfile,
-      "coinlib_build_secp256k1_linux",
-      "output/libsecp256k1.so",
+    cmd,
+    dockerfile,
+    "coinlib_build_secp256k1_linux",
+    "output/libsecp256k1.so",
   )) {
     exit(1);
   }
-
 }

coinlib/bin/build_macos.dart

@@ -4,33 +4,34 @@ import 'util.dart';
 /// Build a universal macOS framework for secp256k1 directly on a mac machine
 
 void main() async {
-
   // Clone secp256k1 to temporary directory to keep source clean
 
   final tmpDir = createTmpDir();
   print("Building in $tmpDir");
-  final libDir = "$tmpDir/secp256k1";
+  final libDir = "$tmpDir/secp256k1-coinlib";
 
   exitOnCode(
     await execWithStdio(
-      "git", ["clone", "https://github.com/bitcoin-core/secp256k1", libDir],
+      "git",
+      ["clone", "https://github.com/peercoin/secp256k1-coinlib", libDir],
     ),
-    "Could not clone secp256k1 to temporary build directory",
+    "Could not clone secp256k1-coinlib to temporary build directory",
   );
 
-  // Checkout to 0.5.0 commit
+  // Checkout to 0.7.0 commit
   exitOnCode(
     await execWithStdio(
-      "git", ["checkout", "e3a885d42a7800c1ccebad94ad1e2b82c4df5c65"],
+      "git",
+      ["checkout", "69018e5b939d8d540ca6b237945100f4ecb5681e"],
       workingDir: libDir,
     ),
-    "Could not checkout to v0.5.0 commit",
+    "Could not checkout to v0.7.0 commit",
   );
 
-  // Generate configure
+  // Generate configure. Do not move to cmake on macOS yet
   exitCode = await execWithStdio("sh", ["./autogen.sh"], workingDir: libDir);
   if (exitCode != 0) {
-    print("Couldn't generate configure for secp256k1");
+    print("Couldn't generate configure for secp256k1-coinlib");
     exit(1);
   }
 
@@ -48,14 +49,14 @@ void main() async {
     workingDir: libDir,
   );
   if (exitCode != 0) {
-    print("Failed to configure secp256k1");
+    print("Failed to configure secp256k1-coinlib");
     exit(1);
   }
 
   // Run make
   exitCode = await execWithStdio("make", [], workingDir: libDir);
   if (exitCode != 0) {
-    print("Failed to make secp256k1");
+    print("Failed to make secp256k1-coinlib");
     exit(1);
   }
 
@@ -69,8 +70,7 @@ void main() async {
   // Copy framework to build directory
   final buildDir = "${Directory.current.path}/build";
   Directory(buildDir).create();
-  final libFile = File("$libDir/build/lib/libsecp256k1.2.dylib");
+  final libFile = File("$libDir/build/lib/libsecp256k1.6.dylib");
   await libFile.copy("$buildDir/libsecp256k1.dylib");
   print("Created dylib in build directory");
-
 }

coinlib/bin/build_wasm.dart

@@ -10,37 +10,42 @@ String dockerfile = r"""
 FROM debian:bookworm
 
 # Install dependenices
-RUN apt-get update -y \
-  && apt-get install -y autoconf libtool build-essential git wget
+RUN apt-get update -y && apt-get install -y cmake git wget
 
 # Download and install wasi-sdk
 
-ENV WASI_VERSION=19
+ENV WASI_VERSION=27
 ENV WASI_VERSION_FULL=${WASI_VERSION}.0
-ENV WASI_SDK_PATH=/wasi-sdk-${WASI_VERSION_FULL}
-ENV WASI_ARCHIVE=wasi-sdk-${WASI_VERSION_FULL}-linux.tar.gz
+ENV WASI_SDK_PATH=/wasi-sdk-${WASI_VERSION_FULL}-x86_64-linux
+ENV WASI_ARCHIVE=wasi-sdk-${WASI_VERSION_FULL}-x86_64-linux.tar.gz
 
 RUN wget -nv https://github.com/WebAssembly/wasi-sdk/releases/download/wasi-sdk-${WASI_VERSION}/$WASI_ARCHIVE
 RUN tar xvf $WASI_ARCHIVE
 RUN rm $WASI_ARCHIVE
 
-# Clone libsecp256k1 and use v0.5.0
-RUN git clone https://github.com/bitcoin-core/secp256k1 \
-  && cd secp256k1 \
-  && git checkout e3a885d42a7800c1ccebad94ad1e2b82c4df5c65
-WORKDIR /secp256k1
+# Clone secp256k1-coinlib and use v0.7.0
+RUN git clone https://github.com/peercoin/secp256k1-coinlib \
+  && cd secp256k1-coinlib \
+  && git checkout 69018e5b939d8d540ca6b237945100f4ecb5681e
+WORKDIR /secp256k1-coinlib
 
 # Build using wasi-sdk
-RUN ./autogen.sh
-RUN ./configure \
-  --enable-module-recovery --disable-tests --disable-shared \
-  --disable-exhaustive-tests --disable-benchmark \
-  --with-sysroot=${WASI_SDK_PATH}/share/wasi-sysroot \
-  --host=wasm32 --target=wasm32-unknown-wasi \
-  CFLAGS="-O2 -fPIC" CC=${WASI_SDK_PATH}/bin/clang
-RUN make
+RUN cmake -B build \
+      -DSECP256K1_ENABLE_MODULE_RECOVERY=ON \
+      -DSECP256K1_BUILD_TESTS=OFF \
+      -DSECP256K1_DISABLE_SHARED=ON \
+      -DSECP256K1_BUILD_EXHAUSTIVE_TESTS=OFF \
+      -DSECP256K1_BUILD_BENCHMARK=OFF \
+      -DSECP256K1_BUILD_EXAMPLES=OFF \
+      -DSECP256K1_BUILD_CTIME_TESTS=OFF \
+      -DCMAKE_TOOLCHAIN_FILE=${WASI_SDK_PATH}/share/cmake/wasi-sdk.cmake \
+      -DCMAKE_C_FLAGS="-O2 -fPIC" \
+      -DCMAKE_BUILD_TYPE=Release
+RUN cmake --build build
 
 # Link output with wasi standard library and export required functions
+# wasm-ld is a bit broken as it requires manual inclusion of necessary symbols
+# but it works. Using clang to link would probably be better.
 RUN mkdir output
 RUN ${WASI_SDK_PATH}/bin/wasm-ld \
   -o output/secp256k1.wasm \
@@ -69,18 +74,33 @@ RUN ${WASI_SDK_PATH}/bin/wasm-ld \
   --export secp256k1_ec_seckey_negate \
   --export secp256k1_keypair_create \
   --export secp256k1_xonly_pubkey_parse \
+  --export secp256k1_xonly_pubkey_serialize \
   --export secp256k1_schnorrsig_sign32 \
   --export secp256k1_schnorrsig_verify \
   --export secp256k1_ecdh \
-  # The secp256k1 library object files
-  src/libsecp256k1_la-secp256k1.o \
-  src/libsecp256k1_precomputed_la-precomputed_ecmult.o \
-  src/libsecp256k1_precomputed_la-precomputed_ecmult_gen.o \
+  --export secp256k1_ec_pubkey_sort \
+  --export secp256k1_musig_pubkey_agg \
+  --export secp256k1_musig_pubkey_xonly_tweak_add \
+  --export secp256k1_musig_nonce_gen \
+  --export secp256k1_musig_pubnonce_parse \
+  --export secp256k1_musig_pubnonce_serialize \
+  --export secp256k1_musig_nonce_agg \
+  --export secp256k1_musig_nonce_process \
+  --export secp256k1_musig_partial_sign \
+  --export secp256k1_musig_partial_sig_parse \
+  --export secp256k1_musig_partial_sig_serialize \
+  --export secp256k1_musig_partial_sig_verify \
+  --export secp256k1_musig_partial_sig_agg \
+  --export secp256k1_musig_nonce_parity \
+  --export secp256k1_musig_adapt \
+  --export secp256k1_musig_extract_adaptor \
+  # The secp256k1 library file
+  build/lib/libsecp256k1.a \
   # Need to include libc for wasi here as it isn't done for us
   ${WASI_SDK_PATH}/share/wasi-sysroot/lib/wasm32-wasi/libc.a \
   # Need to include another library from clang that isn't included either
   # See https://github.com/WebAssembly/wasi-libc/issues/98
-  ${WASI_SDK_PATH}/lib/clang/15.0.7/lib/wasi/libclang_rt.builtins-wasm32.a
+  ${WASI_SDK_PATH}/lib/clang/20/lib/wasm32-unknown-wasi/libclang_rt.builtins.a
 """;
 
 void binaryFileToDart(String inPath, String outPath, String name) {
@@ -93,7 +113,6 @@ final $name = base64Decode("$b64");""";
 }
 
 void main() async {
-
   String cmd = await getDockerCmd();
   print("Using $cmd to run dockerfile");
 
@@ -103,11 +122,11 @@ void main() async {
 
   // Build secp256k1 to wasm and copy wasm file to tempdir
   if (!await dockerRun(
-      cmd,
-      dockerfile,
-      "coinlib_build_secp256k1_wasm",
-      tmpDir,
-      "cp output/secp256k1.wasm /host/secp256k1.wasm",
+    cmd,
+    dockerfile,
+    "coinlib_build_secp256k1_wasm",
+    tmpDir,
+    "cp output/secp256k1.wasm /host/secp256k1.wasm",
   )) {
     exit(1);
   }
@@ -119,5 +138,4 @@ void main() async {
     "secp256k1WasmData",
   );
   print("Output secp256k1.wasm.g.dart successfully");
-
 }