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

WIP: Upgrade the bitcoin dependency #308

Conversation

tcharding
Copy link
Member

@tcharding tcharding commented Aug 29, 2023

Upgrade the bitcoin dependency to the soon-to-be-released version.

Doing this patch lead to:

Once they merge and we release then this PR can come off WIP.

Done on top of bitcoin master branch while on commit: 7fd9b89e Merge rust-bitcoin/rust-bitcoin#2010: Use weight type for stripped_size with the addition of the two PRs above.

Upgrade the `bitcoin` dependency to the soon-to-be-released version.

Doing this patch lead to:

- rust-bitcoin/rust-bitcoin#2038
- rust-bitcoin/rust-bitcoin#2039

Once they merge and we release then this PR can come off WIP.
@lateminer
Copy link

Your PR rust-bitcoin/rust-bitcoin#2039 is supposed to add enum with name DecodeHexError, not FromHexError.

@tcharding
Copy link
Member Author

Thanks @romanz, I did this months ago, I'm not able to remember what happened and why I did #311. I'd say I just forgot that I did this one when I was madly doing all the others (that same day I accidentally came into your crate and updated it :)

@tcharding tcharding closed this Oct 27, 2023
@tcharding
Copy link
Member Author

FTR rust-bitcoin/rust-bitcoin#2039 won't go in the rust-bitcoin v0.31.0 release.

lateminer added a commit to CoinBlack/rust-blackcoin that referenced this pull request Oct 27, 2023
As this function won't be included in `rust-bitcoin v0.31.0` release according to  rust-bitcoin/rust-bitcoincore-rpc#308 (comment)
@tcharding tcharding deleted the 08-29-upgrade-bitcoin-dependency branch May 1, 2024 02:29
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.

None yet

2 participants