HomePhabricator

[electrum] fix the btchip-python dependency for pip > 24.1

Description

[electrum] fix the btchip-python dependency for pip > 24.1

Summary:
The latest btchip-python release contained a malformed dependency that used to be treated as a warning, but is now treated as an error by recent pip version. And even with the slightly older versions of pip currently used in release scripts we already get an error the Windows build, because there is an additional step (compared with the linux appimage build) that already treats this as an error.

It seems like btchip-python is no longer maintained by Ledger, and the proposed fix in https://github.com/LedgerHQ/btchip-python/pull/54 is not going to be merged and released.

This diff changes the requirements file to install a version including that patch, downloaded from github's archive API

Depends on D16667

Test Plan:
Checked that the fix works.

python -m venv test-env
source test-env/bin/activate

pip install pip --upgrade
pip --version
pip install -r ~/dev/bitcoin-abc/electrum/contrib/deterministic-build/requirements-binaries.txt
pip install -r ~/dev/bitcoin-abc/electrum/contrib/deterministic-build/requirements-hw.txt
deactivate

Checked that the Windows release script works

contrib/build-wine/build.sh

Checked that the patch commit is based on the latest commit of https://github.com/LedgerHQ/btchip-python/commits/master/ (tag 0.1.32 + 2 commits that fix Fedora packaging).

Checked the archive's sha256sum.

Reviewers: #bitcoin_abc, roqqit

Reviewed By: roqqit

Differential Revision: https://reviews.bitcoinabc.org/D16669

Details

Provenance
PiRKAuthored on Aug 21 2024, 08:20
PiRKPushed on Aug 26 2024, 18:59
Reviewer
roqqit
Differential Revision
D16669: [electrum] fix the btchip-python dependency for pip > 24.1
Parents
rABCf35fef42cb80: [electrum] update dockerfiles to unbreak the release build
Branches
Unknown
Tags
Unknown