[0/5] Add ledger hardware wallet support

  • Open
  • quality assurance status badge
Details
3 participants
  • Andreas Enge
  • jesse
  • James Smith
Owner
unassigned
Submitted by
jesse
Severity
normal
J
J
jesse wrote 4 days ago
(address . guix-patches@gnu.org)(name . Jesse Eisses)(address . jesse@eisses.email)
cover.1729985174.git.jesse@eisses.email
From: Jesse Eisses <jesse@eisses.email>

This patch adds ledger hardware wallet support to electrum.

Several additional Python packages are needed this:

After adding the dependencies to electrum it's possible to access ledger
within electrum.

Jesse Eisses (5):
gnu: Add python-ledgercomm
gnu: Add python-coincurve
gnu: Add python-bip32
gnu: Add python-ledger-bitcoin
gnu: Add ledger hardware wallet support to electrum

gnu/packages/finance.scm | 99 ++++++++++++++++++++++++++++++++++++++++
1 file changed, 99 insertions(+)


base-commit: 269e4034fcaf55324187efffb6ed5ba14d5e9286
--
2.46.0
J
J
jesse wrote 4 days ago
[1/5] gnu: Add python-ledgercomm
(address . 74039@debbugs.gnu.org)(name . Jesse Eisses)(address . jesse@eisses.email)
4daddcdedef78a17701759dd722dbea938e370cc.1729985174.git.jesse@eisses.email
From: Jesse Eisses <jesse@eisses.email>

Change-Id: If62d5425c346063787da81a47907f5ee84a749ab
---
gnu/packages/finance.scm | 18 ++++++++++++++++++
1 file changed, 18 insertions(+)

Toggle diff (31 lines)
diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
index d4594a1083..59a11e6925 100644
--- a/gnu/packages/finance.scm
+++ b/gnu/packages/finance.scm
@@ -1100,6 +1100,24 @@ (define-public python-btchip-python
Nano dongle.")
(license license:asl2.0)))
+(define-public python-ledgercomm
+ (package
+ (name "python-ledgercomm")
+ (version "1.2.0")
+ (source (origin
+ (method url-fetch)
+ (uri (pypi-uri "ledgercomm" version))
+ (sha256
+ (base32
+ "019f13h59yxxhyhvi5xz8b20pfhazbvn9sgx5x98mp2shj6cks8y"))))
+ (arguments
+ (list #:tests? #f))
+ (build-system pyproject-build-system)
+ (home-page "https://github.com/LedgerHQ/ledgercomm")
+ (synopsis "Library to communicate with Ledger Nano S/X and Speculos")
+ (description "Library to communicate with Ledger Nano S/X and Speculos")
+ (license license:expat)))
+
(define-public python-trezor
(package
(name "python-trezor")
--
2.46.0
J
J
jesse wrote 4 days ago
[2/5] gnu: Add python-coincurve
(address . 74039@debbugs.gnu.org)(name . Jesse Eisses)(address . jesse@eisses.email)
bda6ecdbf4385edd3012abceddb5183c4ca1263a.1729985174.git.jesse@eisses.email
From: Jesse Eisses <jesse@eisses.email>

Change-Id: I2acb09def9193247b1e4169321cf22bdb77c9960
---
gnu/packages/finance.scm | 30 ++++++++++++++++++++++++++++++
1 file changed, 30 insertions(+)

Toggle diff (50 lines)
diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
index 59a11e6925..c3ff34e0cc 100644
--- a/gnu/packages/finance.scm
+++ b/gnu/packages/finance.scm
@@ -109,6 +109,7 @@ (define-module (gnu packages finance)
#:use-module (gnu packages jemalloc)
#:use-module (gnu packages libedit)
#:use-module (gnu packages libevent)
+ #:use-module (gnu packages libffi)
#:use-module (gnu packages libunwind)
#:use-module (gnu packages libusb)
#:use-module (gnu packages linux)
@@ -1118,6 +1119,35 @@ (define-public python-ledgercomm
(description "Library to communicate with Ledger Nano S/X and Speculos")
(license license:expat)))
+(define-public python-coincurve
+ (package
+ (name "python-coincurve")
+ (version "18.0.0")
+ (source (origin
+ (method url-fetch)
+ (uri (pypi-uri "coincurve" version))
+ (sha256
+ (base32
+ "1hagr7hicfc1hkg5lfxj0di53bp3rbpq0rz5h279v80pwjpjcrn8"))))
+ (build-system pyproject-build-system)
+ (propagated-inputs (list python-asn1crypto python-cffi))
+ (home-page "https://pypi.org/project/coincurve")
+ (inputs (list libsecp256k1))
+ (native-inputs (list autoconf automake libtool pkg-config))
+ (arguments
+ (list #:phases
+ #~(modify-phases %standard-phases
+ (delete 'check)
+ (add-after 'unpack 'use-libsecp256k1-input
+ (lambda* (#:key inputs #:allow-other-keys)
+ (let ((lib-dir (string-append
+ (assoc-ref inputs "libsecp256k1") "/lib")))
+ (setenv "LIB_DIR" lib-dir)))))))
+ (synopsis "Cross-platform Python CFFI bindings for libsecp256k1")
+ (description "Cross-platform Python CFFI bindings for libsecp256k1")
+ (license (list license:asl2.0
+ license:expat))))
+
(define-public python-trezor
(package
(name "python-trezor")
--
2.46.0
J
J
jesse wrote 4 days ago
[5/5] gnu: Add ledger hardware wallet support to electrum
(address . 74039@debbugs.gnu.org)(name . Jesse Eisses)(address . jesse@eisses.email)
19e0e5b28f2a18f838d7cb6819385812facc9501.1729985174.git.jesse@eisses.email
From: Jesse Eisses <jesse@eisses.email>

Change-Id: I348533df6611a47dca5fd933eed19cebdfca025d
---
gnu/packages/finance.scm | 4 ++++
1 file changed, 4 insertions(+)

Toggle diff (33 lines)
diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
index 317d4592f3..84373452c4 100644
--- a/gnu/packages/finance.scm
+++ b/gnu/packages/finance.scm
@@ -132,6 +132,7 @@ (define-module (gnu packages finance)
#:use-module (gnu packages qt)
#:use-module (gnu packages readline)
#:use-module (gnu packages security-token)
+ #:use-module (gnu packages serialization)
#:use-module (gnu packages sphinx)
#:use-module (gnu packages sqlite)
#:use-module (gnu packages tex)
@@ -579,15 +580,18 @@ (define-public electrum
python-attrs
python-bitstring
python-btchip-python
+ python-cbor
python-certifi
python-cryptography
python-dnspython
python-hidapi
python-ledgerblue
+ python-ledger-bitcoin
python-protobuf
python-pyqt
python-qdarkstyle
python-qrcode
+ libusb
zbar))
(arguments
`(#:tests? #f ; no tests
--
2.46.0
J
J
jesse wrote 4 days ago
[3/5] gnu: Add python-bip32
(address . 74039@debbugs.gnu.org)(name . Jesse Eisses)(address . jesse@eisses.email)
f8e78b4ebd7786414b2c0b51ac157200b880df35.1729985174.git.jesse@eisses.email
From: Jesse Eisses <jesse@eisses.email>

Change-Id: I6f6e5a2048d9d124808bc53df9b493e61c0bcf07
---
gnu/packages/finance.scm | 23 +++++++++++++++++++++++
1 file changed, 23 insertions(+)

Toggle diff (36 lines)
diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
index c3ff34e0cc..2711987673 100644
--- a/gnu/packages/finance.scm
+++ b/gnu/packages/finance.scm
@@ -1148,6 +1148,29 @@ (define-public python-coincurve
(license (list license:asl2.0
license:expat))))
+(define-public python-bip32
+ (package
+ (name "python-bip32")
+ (version "3.4")
+ (source (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/darosior/python-bip32")
+ (commit "1492d39312f1d9630363c292f6ab8beb8ceb16dd")))
+ (sha256
+ (base32
+ "048zwh1i11jbs6h8j8sqbl7kx5hcyhr7i61mq5snl3fpbr3hmid3"))))
+ (build-system pyproject-build-system)
+ (propagated-inputs (list python-base58 python-coincurve))
+ (inputs (list gnu-make))
+ (arguments
+ (list #:tests? #f))
+ (home-page "https://github.com/darosior/python-bip32")
+ (synopsis "Minimalistic implementation of the BIP32 key derivation scheme")
+ (description
+ "Minimalistic implementation of the BIP32 key derivation scheme")
+ (license license:expat)))
+
(define-public python-trezor
(package
(name "python-trezor")
--
2.46.0
J
J
jesse wrote 4 days ago
[4/5] gnu: Add python-ledger-bitcoin
(address . 74039@debbugs.gnu.org)(name . Jesse Eisses)(address . jesse@eisses.email)
6c715b37ad1a81f269b62581ed7f514c48a09f82.1729985174.git.jesse@eisses.email
From: Jesse Eisses <jesse@eisses.email>

Change-Id: Ic7956e573712e478981de47466468f339b153741
---
gnu/packages/finance.scm | 24 ++++++++++++++++++++++++
1 file changed, 24 insertions(+)

Toggle diff (37 lines)
diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
index 2711987673..317d4592f3 100644
--- a/gnu/packages/finance.scm
+++ b/gnu/packages/finance.scm
@@ -1171,6 +1171,30 @@ (define-public python-bip32
"Minimalistic implementation of the BIP32 key derivation scheme")
(license license:expat)))
+(define-public python-ledger-bitcoin
+ (package
+ (name "python-ledger-bitcoin")
+ (version "0.2.0")
+ (source (origin
+ (method url-fetch)
+ (uri (pypi-uri "ledger_bitcoin" version))
+ (sha256
+ (base32
+ "1drd15h4vsr5ysrmrrhh2lalzmnxkvi82x6a433yr8c2vf6zsynx"))))
+ (build-system pyproject-build-system)
+ (arguments
+ (list #:tests? #f
+ #:phases
+ #~(modify-phases %standard-phases
+ (delete 'sanity-check))))
+ (propagated-inputs (list python-bip32 python-coincurve python-ledgercomm
+ python-packaging python-typing-extensions))
+ (inputs (list python-ledgercomm))
+ (home-page "https://github.com/LedgerHQ/app-bitcoin-new")
+ (synopsis "Client for Ledger Nano Bitcoin application")
+ (description "Client for Ledger Nano Bitcoin application")
+ (license license:asl2.0)))
+
(define-public python-trezor
(package
(name "python-trezor")
--
2.46.0
J
J
Jesse Eisses wrote 4 days ago
[0/5] Add ledger hardware wallet support
(address . 74039@debbugs.gnu.org)
87wmhuvuji.fsf@eisses.email
Hi there,

I just realize that I forgot to add my name to the list of contributors,
and also to prefix the email subject with PATCH. Apologies, still
getting the hang of this ?

Not sure how to fix this. Should I submit a whole new patch series to this
thread, or even start a new issue with the PATCH prefix?

Thanks,

Jesse

jesse@eisses.email writes:

Toggle quote (38 lines)
> From: Jesse Eisses <jesse@eisses.email>
>
> Change-Id: I348533df6611a47dca5fd933eed19cebdfca025d
> ---
> gnu/packages/finance.scm | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
> index 317d4592f3..84373452c4 100644
> --- a/gnu/packages/finance.scm
> +++ b/gnu/packages/finance.scm
> @@ -132,6 +132,7 @@ (define-module (gnu packages finance)
> #:use-module (gnu packages qt)
> #:use-module (gnu packages readline)
> #:use-module (gnu packages security-token)
> + #:use-module (gnu packages serialization)
> #:use-module (gnu packages sphinx)
> #:use-module (gnu packages sqlite)
> #:use-module (gnu packages tex)
> @@ -579,15 +580,18 @@ (define-public electrum
> python-attrs
> python-bitstring
> python-btchip-python
> + python-cbor
> python-certifi
> python-cryptography
> python-dnspython
> python-hidapi
> python-ledgerblue
> + python-ledger-bitcoin
> python-protobuf
> python-pyqt
> python-qdarkstyle
> python-qrcode
> + libusb
> zbar))
> (arguments
> `(#:tests? #f ; no tests
J
J
James Smith wrote 3 days ago
(name . Jesse Eisses via Guix-patches via)(address . guix-patches@gnu.org)
86cyjleino.fsf@disroot.org
Jesse Eisses via Guix-patches via <guix-patches@gnu.org> writes:

Toggle quote (13 lines)
> Hi there,
>
> I just realize that I forgot to add my name to the list of contributors,
> and also to prefix the email subject with PATCH. Apologies, still
> getting the hang of this ?
>
> Not sure how to fix this. Should I submit a whole new patch series to this
> thread, or even start a new issue with the PATCH prefix?
>
> Thanks,
>
> Jesse

Hi Jesse,

You can just add your name to the list, rebase the change to your first
commit, and resend this patch series to this issue with a v2. For
example, my workflow with Emacs and Magit is as follows:

1. Add copyright to gnu/packages/finance.scm after the last one.
2. Using Magit, stage your changes, press "c" for commit, followed by "F"
for instant fixup, the select the commit to apply the changes to (I.E.
for copyright, it'll typically be patch 1/5).
3. Once you're done making changes, use "git send-email" to send a new
set of patches. For example, you'll use "git send-email -5 -v2
--to=74039@debbugs.gnu.org".

For more details, you can check the relevant section of the manual at

Hope that helps,
James
J
J
Jesse Eisses wrote 3 days ago
(name . James Smith)(address . jsubuntuxp@disroot.org)
87iktdi79p.fsf@eisses.email
Hi James,

Thanks for that! New patch set is incoming

- Jesse

James Smith <jsubuntuxp@disroot.org> writes:

Toggle quote (34 lines)
> Jesse Eisses via Guix-patches via <guix-patches@gnu.org> writes:
>
>> Hi there,
>>
>> I just realize that I forgot to add my name to the list of contributors,
>> and also to prefix the email subject with PATCH. Apologies, still
>> getting the hang of this ?
>>
>> Not sure how to fix this. Should I submit a whole new patch series to this
>> thread, or even start a new issue with the PATCH prefix?
>>
>> Thanks,
>>
>> Jesse
>
> Hi Jesse,
>
> You can just add your name to the list, rebase the change to your first
> commit, and resend this patch series to this issue with a v2. For
> example, my workflow with Emacs and Magit is as follows:
>
> 1. Add copyright to gnu/packages/finance.scm after the last one.
> 2. Using Magit, stage your changes, press "c" for commit, followed by "F"
> for instant fixup, the select the commit to apply the changes to (I.E.
> for copyright, it'll typically be patch 1/5).
> 3. Once you're done making changes, use "git send-email" to send a new
> set of patches. For example, you'll use "git send-email -5 -v2
> --to=74039@debbugs.gnu.org".
>
> For more details, you can check the relevant section of the manual at
> https://guix.gnu.org/manual/devel/en/html_node/Sending-a-Patch-Series.html
>
> Hope that helps,
> James
J
J
jesse wrote 3 days ago
[PATCH v2 1/5] gnu: Add python-ledgercomm
(address . 74039@debbugs.gnu.org)(name . Jesse Eisses)(address . jesse@eisses.email)
42cbc6b9509ad22d60d78258413b304fc957c886.1730063595.git.jesse@eisses.email
From: Jesse Eisses <jesse@eisses.email>

Change-Id: If62d5425c346063787da81a47907f5ee84a749ab
---
gnu/packages/finance.scm | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)

Toggle diff (41 lines)
diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
index d4594a1083..262380d13e 100644
--- a/gnu/packages/finance.scm
+++ b/gnu/packages/finance.scm
@@ -39,6 +39,7 @@
;;; Copyright © 2023 Attila Lendvai <attila@lendvai.name>
;;; Copyright © 2024 Saku Laesvuori <saku@laesvuori.fi>
;;; Copyright © 2024 Sharlatan Hellseher <sharlatanus@gmail.com>
+;;; Copyright © 2024 Jesse Eisses <jesse@eisses.email>
;;;
;;; This file is part of GNU Guix.
;;;
@@ -1100,6 +1101,24 @@ (define-public python-btchip-python
Nano dongle.")
(license license:asl2.0)))
+(define-public python-ledgercomm
+ (package
+ (name "python-ledgercomm")
+ (version "1.2.0")
+ (source (origin
+ (method url-fetch)
+ (uri (pypi-uri "ledgercomm" version))
+ (sha256
+ (base32
+ "019f13h59yxxhyhvi5xz8b20pfhazbvn9sgx5x98mp2shj6cks8y"))))
+ (arguments
+ (list #:tests? #f))
+ (build-system pyproject-build-system)
+ (home-page "https://github.com/LedgerHQ/ledgercomm")
+ (synopsis "Library to communicate with Ledger Nano S/X and Speculos")
+ (description "Library to communicate with Ledger Nano S/X and Speculos")
+ (license license:expat)))
+
(define-public python-trezor
(package
(name "python-trezor")

base-commit: 269e4034fcaf55324187efffb6ed5ba14d5e9286
--
2.46.0
J
J
jesse wrote 3 days ago
[PATCH v2 2/5] gnu: Add python-coincurve
(address . 74039@debbugs.gnu.org)(name . Jesse Eisses)(address . jesse@eisses.email)
d2d3df469c7e8465ead1d88b31e3e3a4f7b148a1.1730063595.git.jesse@eisses.email
From: Jesse Eisses <jesse@eisses.email>

Change-Id: I2acb09def9193247b1e4169321cf22bdb77c9960
---
gnu/packages/finance.scm | 30 ++++++++++++++++++++++++++++++
1 file changed, 30 insertions(+)

Toggle diff (50 lines)
diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
index 262380d13e..d96055601d 100644
--- a/gnu/packages/finance.scm
+++ b/gnu/packages/finance.scm
@@ -110,6 +110,7 @@ (define-module (gnu packages finance)
#:use-module (gnu packages jemalloc)
#:use-module (gnu packages libedit)
#:use-module (gnu packages libevent)
+ #:use-module (gnu packages libffi)
#:use-module (gnu packages libunwind)
#:use-module (gnu packages libusb)
#:use-module (gnu packages linux)
@@ -1119,6 +1120,35 @@ (define-public python-ledgercomm
(description "Library to communicate with Ledger Nano S/X and Speculos")
(license license:expat)))
+(define-public python-coincurve
+ (package
+ (name "python-coincurve")
+ (version "18.0.0")
+ (source (origin
+ (method url-fetch)
+ (uri (pypi-uri "coincurve" version))
+ (sha256
+ (base32
+ "1hagr7hicfc1hkg5lfxj0di53bp3rbpq0rz5h279v80pwjpjcrn8"))))
+ (build-system pyproject-build-system)
+ (propagated-inputs (list python-asn1crypto python-cffi))
+ (home-page "https://pypi.org/project/coincurve")
+ (inputs (list libsecp256k1))
+ (native-inputs (list autoconf automake libtool pkg-config))
+ (arguments
+ (list #:phases
+ #~(modify-phases %standard-phases
+ (delete 'check)
+ (add-after 'unpack 'use-libsecp256k1-input
+ (lambda* (#:key inputs #:allow-other-keys)
+ (let ((lib-dir (string-append
+ (assoc-ref inputs "libsecp256k1") "/lib")))
+ (setenv "LIB_DIR" lib-dir)))))))
+ (synopsis "Cross-platform Python CFFI bindings for libsecp256k1")
+ (description "Cross-platform Python CFFI bindings for libsecp256k1")
+ (license (list license:asl2.0
+ license:expat))))
+
(define-public python-trezor
(package
(name "python-trezor")
--
2.46.0
J
J
jesse wrote 3 days ago
[PATCH v2 3/5] gnu: Add python-bip32
(address . 74039@debbugs.gnu.org)(name . Jesse Eisses)(address . jesse@eisses.email)
b08d17ea1bfb60780babfaa26605a07a0ac67f86.1730063595.git.jesse@eisses.email
From: Jesse Eisses <jesse@eisses.email>

Change-Id: I6f6e5a2048d9d124808bc53df9b493e61c0bcf07
---
gnu/packages/finance.scm | 23 +++++++++++++++++++++++
1 file changed, 23 insertions(+)

Toggle diff (36 lines)
diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
index d96055601d..7e030daca9 100644
--- a/gnu/packages/finance.scm
+++ b/gnu/packages/finance.scm
@@ -1149,6 +1149,29 @@ (define-public python-coincurve
(license (list license:asl2.0
license:expat))))
+(define-public python-bip32
+ (package
+ (name "python-bip32")
+ (version "3.4")
+ (source (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/darosior/python-bip32")
+ (commit "1492d39312f1d9630363c292f6ab8beb8ceb16dd")))
+ (sha256
+ (base32
+ "048zwh1i11jbs6h8j8sqbl7kx5hcyhr7i61mq5snl3fpbr3hmid3"))))
+ (build-system pyproject-build-system)
+ (propagated-inputs (list python-base58 python-coincurve))
+ (inputs (list gnu-make))
+ (arguments
+ (list #:tests? #f))
+ (home-page "https://github.com/darosior/python-bip32")
+ (synopsis "Minimalistic implementation of the BIP32 key derivation scheme")
+ (description
+ "Minimalistic implementation of the BIP32 key derivation scheme")
+ (license license:expat)))
+
(define-public python-trezor
(package
(name "python-trezor")
--
2.46.0
J
J
jesse wrote 3 days ago
[PATCH v2 4/5] gnu: Add python-ledger-bitcoin
(address . 74039@debbugs.gnu.org)(name . Jesse Eisses)(address . jesse@eisses.email)
65c5519b3090d2f5a40eb1341e7f2cbdc68c1937.1730063596.git.jesse@eisses.email
From: Jesse Eisses <jesse@eisses.email>

Change-Id: Ic7956e573712e478981de47466468f339b153741
---
gnu/packages/finance.scm | 24 ++++++++++++++++++++++++
1 file changed, 24 insertions(+)

Toggle diff (37 lines)
diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
index 7e030daca9..4be438a6d6 100644
--- a/gnu/packages/finance.scm
+++ b/gnu/packages/finance.scm
@@ -1172,6 +1172,30 @@ (define-public python-bip32
"Minimalistic implementation of the BIP32 key derivation scheme")
(license license:expat)))
+(define-public python-ledger-bitcoin
+ (package
+ (name "python-ledger-bitcoin")
+ (version "0.2.0")
+ (source (origin
+ (method url-fetch)
+ (uri (pypi-uri "ledger_bitcoin" version))
+ (sha256
+ (base32
+ "1drd15h4vsr5ysrmrrhh2lalzmnxkvi82x6a433yr8c2vf6zsynx"))))
+ (build-system pyproject-build-system)
+ (arguments
+ (list #:tests? #f
+ #:phases
+ #~(modify-phases %standard-phases
+ (delete 'sanity-check))))
+ (propagated-inputs (list python-bip32 python-coincurve python-ledgercomm
+ python-packaging python-typing-extensions))
+ (inputs (list python-ledgercomm))
+ (home-page "https://github.com/LedgerHQ/app-bitcoin-new")
+ (synopsis "Client for Ledger Nano Bitcoin application")
+ (description "Client for Ledger Nano Bitcoin application")
+ (license license:asl2.0)))
+
(define-public python-trezor
(package
(name "python-trezor")
--
2.46.0
J
J
jesse wrote 3 days ago
[PATCH v2 5/5] gnu: Add ledger hardware wallet support to electrum
(address . 74039@debbugs.gnu.org)(name . Jesse Eisses)(address . jesse@eisses.email)
07bd766f7763ba317c5577833d7bd56ee4643dae.1730063596.git.jesse@eisses.email
From: Jesse Eisses <jesse@eisses.email>

Change-Id: I348533df6611a47dca5fd933eed19cebdfca025d
---
gnu/packages/finance.scm | 4 ++++
1 file changed, 4 insertions(+)

Toggle diff (33 lines)
diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
index 4be438a6d6..f6fcaedef2 100644
--- a/gnu/packages/finance.scm
+++ b/gnu/packages/finance.scm
@@ -133,6 +133,7 @@ (define-module (gnu packages finance)
#:use-module (gnu packages qt)
#:use-module (gnu packages readline)
#:use-module (gnu packages security-token)
+ #:use-module (gnu packages serialization)
#:use-module (gnu packages sphinx)
#:use-module (gnu packages sqlite)
#:use-module (gnu packages tex)
@@ -580,15 +581,18 @@ (define-public electrum
python-attrs
python-bitstring
python-btchip-python
+ python-cbor
python-certifi
python-cryptography
python-dnspython
python-hidapi
python-ledgerblue
+ python-ledger-bitcoin
python-protobuf
python-pyqt
python-qdarkstyle
python-qrcode
+ libusb
zbar))
(arguments
`(#:tests? #f ; no tests
--
2.46.0
A
A
Andreas Enge wrote 3 days ago
Comments
(address . 74039@debbugs.gnu.org)
Zx9a3nfcGSZeXG_a@jurong
Hello Jesse,

thanks for your contribution, which passes QA and looks mainly good!

QA runs "guix lint":
which complains about the following:
"the source file name should contain the package name"

Maybe you could run "guix lint" on your different packages and see to
which one it applies? Probably python-bip32 with its source taken from git.

Maybe you could use pypi there as well? When trying a "guix import pypi bip32"
I get a version 4.0 instead of 3.2; would this work as well?
Otherwise, there are examples in the Guix source of how to force a
descriptive name for the git checkout.

The description should consist of complete sentences and be a bit longer
and more descriptive, maybe a short paragraph. Often it can be taken from
the project website, after stripping out the marketing speech.

Please also adapt the commit messages to the Guix style.
Running "./etc/committer.scm" (instead of "git commit") could be an
option, it usually works well for adding new packages.

Looking forward to a v3,

Andreas
?
Your comment

Commenting via the web interface is currently disabled.

To comment on this conversation send an email to 74039@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 74039
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch