Skip to content
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.

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

jade: update Jade api to 1.0.33 #768

Merged
merged 2 commits into from
Dec 11, 2024

Conversation

JamieDriver
Copy link
Contributor

This extends the serial api to recognise recently supported hardware.

@JamieDriver JamieDriver mentioned this pull request Dec 5, 2024
…m repo

Do not clone submodules recursively as an unused upstream module has
disappeared, and the qemu build handles fetching submodules as needed.
Also add flag to clone submodules without their entire history.
This extends the serial api to recognise recently supported hardware.
@achow101
Copy link
Member

ACK f4f7fe5

@achow101 achow101 merged commit 3174515 into bitcoin-core:master Dec 11, 2024
196 of 286 checks passed
@JamieDriver JamieDriver deleted the update_jade_api branch December 12, 2024 09:10
achow101 added a commit that referenced this pull request Jan 3, 2025
c66dd1a jade: use Jade's native PSBT signing if the firmware version supports it (Jamie C. Driver)

Pull request description:

  1. latest jade api to recognise more recently added hw (now available as its own PR, #768)
  2. remove code which walked psbt and mapped it into Jade's legacy 'sign_tx' json format - instead pass PSBT directly to Jade for signing, since this has been supported in fw now for well over a year and seems to be working well.

ACKs for top commit:
  achow101:
    ACK c66dd1a

Tree-SHA512: e6c80f7309995ba4bf7cc5f9185ad89d45e84226d05cc0ec8741b3a9eead3bda7f4d99e94a16041c6acb2a103719f64f4c2879ed79be6d901cbac4c13ffb689b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants