Skip to content

Conversation

@benma
Copy link
Contributor

@benma benma commented Oct 16, 2025

@benma benma requested a review from NickeZ October 16, 2025 13:21
@benma benma force-pushed the op-return-support branch from b688940 to dff44c2 Compare October 16, 2025 13:27
@NickeZ
Copy link
Collaborator

NickeZ commented Oct 16, 2025

I'm going to need some instructions on how to test this PR :)

@benma
Copy link
Contributor Author

benma commented Oct 16, 2025

I'm going to need some instructions on how to test this PR :)

Hmm you didn't test the same PR in Go 😄 BitBoxSwiss/bitbox02-api-go#141

There is a simulator test and unit tests here, isn't that sufficient? I could try to provide a new file in examples/, but it's not so great to keep adding more examples there 🤔

@benma
Copy link
Contributor Author

benma commented Oct 16, 2025

@NickeZ added an example anyway, run make example-btc-sign-op-return, see added commit.

@NickeZ
Copy link
Collaborator

NickeZ commented Oct 17, 2025

@NickeZ added an example anyway, run make example-btc-sign-op-return, see added commit.

$ make example-btc-sign-op-return
cargo run --example btc_sign_op_return --features=usb,tokio/rt,tokio/macros
   Compiling bitcoin-internals v0.3.0
   Compiling secp256k1-sys v0.10.0
   Compiling serde v1.0.203
   Compiling bitcoin_hashes v0.14.0
   Compiling getrandom v0.2.15
   Compiling bitcoin v0.32.2
   Compiling hex v0.4.3
   Compiling rand_core v0.6.4
   Compiling x25519-dalek v2.0.1
   Compiling noise-rust-crypto v0.6.2
   Compiling base58ck v0.1.0
   Compiling bitcoin-units v0.1.2
   Compiling serde_json v1.0.120
   Compiling serde_urlencoded v0.7.1
   Compiling reqwest v0.12.5
   Compiling secp256k1 v0.29.0
   Compiling miniscript v12.0.0
   Compiling bitbox-api v0.11.0 (/Users/niklasdusenlund/projects/shift/bitbox-api-rs)
error[E0599]: no method named `finalize_mut` found for struct `bitcoin::Psbt` in the current scope
   --> examples/btc_sign_op_return.rs:165:10
    |
165 |     psbt.finalize_mut(&secp).unwrap();
    |          ^^^^^^^^^^^^ method not found in `bitcoin::Psbt`

For more information about this error, try `rustc --explain E0599`.
error: could not compile `bitbox-api` (example "btc_sign_op_return") due to 1 previous error
make: *** [example-btc-sign-op-return] Error 101

@benma benma force-pushed the op-return-support branch from fa51aa6 to e0880cf Compare October 17, 2025 12:25
@benma
Copy link
Contributor Author

benma commented Oct 17, 2025

@NickeZ added an example anyway, run make example-btc-sign-op-return, see added commit.

$ make example-btc-sign-op-return
cargo run --example btc_sign_op_return --features=usb,tokio/rt,tokio/macros
   Compiling bitcoin-internals v0.3.0
   Compiling secp256k1-sys v0.10.0
   Compiling serde v1.0.203
   Compiling bitcoin_hashes v0.14.0
   Compiling getrandom v0.2.15
   Compiling bitcoin v0.32.2
   Compiling hex v0.4.3
   Compiling rand_core v0.6.4
   Compiling x25519-dalek v2.0.1
   Compiling noise-rust-crypto v0.6.2
   Compiling base58ck v0.1.0
   Compiling bitcoin-units v0.1.2
   Compiling serde_json v1.0.120
   Compiling serde_urlencoded v0.7.1
   Compiling reqwest v0.12.5
   Compiling secp256k1 v0.29.0
   Compiling miniscript v12.0.0
   Compiling bitbox-api v0.11.0 (/Users/niklasdusenlund/projects/shift/bitbox-api-rs)
error[E0599]: no method named `finalize_mut` found for struct `bitcoin::Psbt` in the current scope
   --> examples/btc_sign_op_return.rs:165:10
    |
165 |     psbt.finalize_mut(&secp).unwrap();
    |          ^^^^^^^^^^^^ method not found in `bitcoin::Psbt`

For more information about this error, try `rustc --explain E0599`.
error: could not compile `bitbox-api` (example "btc_sign_op_return") due to 1 previous error
make: *** [example-btc-sign-op-return] Error 101

Oops, forgot to delete these lines. Removed now

Copy link
Collaborator

@NickeZ NickeZ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tACK

@benma benma merged commit 2890c25 into BitBoxSwiss:master Oct 30, 2025
3 checks passed
@benma benma deleted the op-return-support branch October 30, 2025 14:04
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