Skip to content

Conversation

@apoelstra
Copy link
Member

We cannot build releases with the existing lockfile because its MSRV is too high. If we intend the MSRV to be lower then we need to fix the lockfile and reduce the MSRV in cargo-rbmt.

@apoelstra
Copy link
Member Author

Actually even with 1.74 the dependencies listed in Cargo.lock don't work. I'll spend some time trying to fix this.

We cannot build `releases` with the existing lockfile because its MSRV
is too high. If we intend the MSRV to be lower then we need to fix the
lockfile and reduce the MSRV in `cargo-rbmt`.
We are very close to being formatted, so just run the formatter to get
us all the way.
@nyonson
Copy link
Collaborator

nyonson commented Dec 4, 2025

@tcharding what is the status of the releases crate? I should have tested it when I made this a workspace, but is it easier now to just fold it in as a subcommand of rbmt?

Copy link
Member Author

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

On 64532c3 successfully ran local tests

@apoelstra
Copy link
Member Author

Personally I think we should fold it into rbmt.

Copy link
Collaborator

@nyonson nyonson left a comment

Choose a reason for hiding this comment

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

ACK 64532c3

@apoelstra apoelstra merged commit 2b52bd8 into rust-bitcoin:master Dec 4, 2025
1 check passed
@apoelstra apoelstra deleted the 2025-12/msrv branch December 4, 2025 21:50
@tcharding
Copy link
Member

Fold it in. FTR I have not looked at it since it was created (I don't even remember writing it if it was me).

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.

3 participants