Skip to content

Even more CI fixes #812

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
achow101 wants to merge 15 commits into
base: master
Choose a base branch
from
Draft

Even more CI fixes #812

achow101 wants to merge 15 commits into from

Conversation

@achow101
Copy link
Member

@achow101 achow101 commented Oct 7, 2025

A number of things changed recently, breaking CI again.

  • Trezor switched from poetry to uv for dependency management, so simulators could not be built
  • There's a rust nightly change that also causes the Trezor simulators to not build that needs patching
  • Ledger's Speculos dropped support for the Nano S which we were using for testing with the Bitcoin Legacy app. We now build the legacy app ourselves and target the Nano S+ in those tests
  • Ledger added new UI elements which need new automation rules for the tests to pass
  • Coldcard sim builder was timing out on clone because of the lwip repo stalling on download. The clone is changed to be a shallow clone along with using the github mirror of lwip.
  • Coldcard patches need to be updated again and a patch in the Coldcard repo needs to be applied to build the simulator
  • Coldcard requires additional dependencies in the builder
  • CI was not properly running the dist tests
  • The tests disabled in Fix CI #795 are re-enabled.

Coldcard is still failing some signing tests, looking into those.

achow101 added 15 commits October 3, 2025 17:06
@achow101
test, coldcard: Update multisig patch
c070643
@achow101
test, trezor: Change to uv
11ade3a 
Trezor switched to uv for dependency management
@achow101
test, trezor: Patch Trezor T to build
09080f2
@achow101
ci: Stop using ubuntu-22.04
8e63078
@achow101
ci: Fix dist tests
18b747e
@achow101
ci, ledger: Build Bitcoin Legacy app and use nanosp for testing it
54b8869 
Recent versions of Speculos no longer support the Nano S. Additionally,
we build the legacy app for the Nano S separately now.
@achow101
ci: Use github mirror of lwip
d4f1873 
lwip is a submodule of many device firmware repos, but the default URLs
often fail to be cloned. Rewrite those URLs with git config to use the
Github mirror to avoid that issue.
@achow101
ci, test: Use shallow clones for device firmware repos
c6c5559
@achow101
test, keepkey: Patch nanopb deprecated file mode
5331398
@achow101
test, ledger: Install speculos dependencies from speculos
1d09f40 
Instead of listing out the dependencies required to install manually,
just install from speculos itself.
@achow101
test, ledger: Update automation
9ba9a17
@achow101
test: Re-enable disabled tests
0fe1dbb
@achow101
test, coldcard: Apply mpy patch before building
73d8564
@achow101
ci: install some more dependencies
5c50a8c
@achow101
test, coldcard: Disable external input tests
a586bc0 
These don't seem to work anymore.
@moneymanolis
Copy link

moneymanolis commented Jan 7, 2026

How about we tag / bring in the maintainers for the projects for which the CI is failing, it's mainly Coldcard apparently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@achow101 @moneymanolis