Skip to content

chore(deps): bump to rust-bitcoin 0.32.5#158

Merged
lucidLuckylee merged 1 commit intoBitVM:mainfrom
alpenlabs:bitcoin-0.32.5
Dec 18, 2024
Merged

chore(deps): bump to rust-bitcoin 0.32.5#158
lucidLuckylee merged 1 commit intoBitVM:mainfrom
alpenlabs:bitcoin-0.32.5

Conversation

@storopoli
Copy link
Copy Markdown
Contributor

@storopoli storopoli commented Dec 16, 2024

Related to rust-bitcoin/rust-bitcoin#3729.

Keeping this as a draft since I am planning to do PRs in more BitVM org crates to bump rust-bitcoin to 0.32.5 (latest).

I had to change one Message deprecation from rust-secp256k1 when updating to 0.29.1 which is the version that rust-bitcoin 0.32.5 re-exports.

Also update musig2 to play nice with [email protected].

Blocked by:

@storopoli storopoli force-pushed the bitcoin-0.32.5 branch 2 times, most recently from 5b2658c to d95729a Compare December 17, 2024 17:23
@lucidLuckylee
Copy link
Copy Markdown
Contributor

@storopoli I merged the blocking PRs.

@storopoli storopoli marked this pull request as ready for review December 18, 2024 07:34
@storopoli
Copy link
Copy Markdown
Contributor Author

This is ready

Comment on lines +73 to +74
let nonce_hash = sha256::Hash::hash(nonce.to_bytes().as_slice());
Message::from_digest_slice(nonce_hash.as_ref()).expect("Failed to create nonce message")
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I decide to .expect() here to not break the API by changing the function signature to return a Result.

@lucidLuckylee
Copy link
Copy Markdown
Contributor

Thank you!

@lucidLuckylee lucidLuckylee merged commit f4a0689 into BitVM:main Dec 18, 2024
@storopoli storopoli deleted the bitcoin-0.32.5 branch December 18, 2024 14:16
wz14 pushed a commit to bitlayer-org/BitVM that referenced this pull request Jan 8, 2025
wz14 pushed a commit to bitlayer-org/BitVM that referenced this pull request Jan 8, 2025
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