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

Updating to bitcoin 0.25 and MSRV 1.29 #152

Merged
merged 1 commit into from
Oct 13, 2020
Merged

Updating to bitcoin 0.25 and MSRV 1.29 #152

merged 1 commit into from
Oct 13, 2020

Conversation

dr-orlovsky
Copy link
Contributor

@dr-orlovsky dr-orlovsky commented Oct 9, 2020

This is trivial, but urgently required for the projects already using bitcoin 0.25 due to secp256k1 library version conflict (you can't use miniscript v2.0 with bitcoin 0.25 in the same crate for now)

@apoelstra
Copy link
Member

It look like the conflict is caused by a bug in rust-secp (seems secp256k1-sys 0.3.0 has link = "rustsecp256k1_v0_2_0" instead of v0_3_0)

@apoelstra
Copy link
Member

We can fix this as part of rust-bitcoin/rust-secp256k1#237 -- once we get that PR merged I'll quickly release secp-sys 0.3.1 and there should no longer be a conflict.

@apoelstra
Copy link
Member

You cannot update bitcoin to 0.25 without also increasing the MSRV of this library.

@dr-orlovsky dr-orlovsky changed the title Updating to bitcoin 0.25 Updating to bitcoin 0.25 and MSRV 1.29 Oct 9, 2020
@dr-orlovsky
Copy link
Contributor Author

@apoelstra MSRV updated as a part of this PR

Copy link
Member

@sanket1729 sanket1729 left a comment

Choose a reason for hiding this comment

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

tAck 7f2482a. Why do we still need the pinning for cc version? It does not seem to work without it.

@dr-orlovsky
Copy link
Contributor Author

I tried to have a build w/o it and it did not pass

@dr-orlovsky
Copy link
Contributor Author

Is it possible to release Miniscript 2.1.0 supporting bitcoin 0.25?

@sanket1729
Copy link
Member

Yes, that is the current plan. We should have the release soon. cc @apoelstra

@apoelstra apoelstra merged commit 3319510 into rust-bitcoin:master Oct 13, 2020
@apoelstra
Copy link
Member

Released 3.0.0 which uses 0.25

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