Page MenuHomePhabricator

[electrum] bump pyinstaller to v6.10.0
ClosedPublic

Authored by PiRK on Sep 6 2024, 07:47.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABCc8ea5bde090d: [electrum] bump pyinstaller to v6.10.0
Summary

We recently bumped setuptools to >= 70.0.0, which forces us to bump pyinstaller (see https://github.com/pyinstaller/pyinstaller/pull/8555)
Fixes an error when attempting to run the generatad windows release.

Unfortunately it does not seem enough to fix the same issue on MacOS, so we also revert the version bump for setuptools for the MacOS build (partial revert of D16481), which fixes the issue.
AFAICT we are not affected by CVE-2024-6345 because we don't use the package_index module of setuptools, so it should be safe to use setuptools < 70.0.

Depends on D16725

Test Plan

Build the MacOS and Windows releases, then check that the executables actually work

Diff Detail

Repository
rABC Bitcoin ABC
Branch
electrum_pyinstaller_bump
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 30204
Build 59935: Build Diffelectrum-tests
Build 59934: arc lint + arc unit

Event Timeline

PiRK requested review of this revision.Sep 6 2024, 07:47
This revision is now accepted and ready to land.Sep 6 2024, 08:03
This revision was automatically updated to reflect the committed changes.