curv icon indicating copy to clipboard operation
curv copied to clipboard

Bump p256 to 0.11

Open Be-P opened this issue 2 years ago • 1 comments

I've bumped the p256 dependency to version 0.11.1 and fixed some compilation issues. See #166

Be-P avatar Jul 22 '22 20:07 Be-P

Please can we land this I want to use the latest version of p256 in dependents and this is blocking me updating as compilation fails with conflicts on ecdsa-core.

Any help much appreciated as always 🙏

Edit: I am using the workaround in #166 for now - thanks @Be-P 👍

tmpfs avatar Aug 13 '22 03:08 tmpfs

Please can we land this I want to use the latest version of p256 in dependents and this is blocking me updating as compilation fails with conflicts on ecdsa-core.

Any help much appreciated as always 🙏

Edit: I am using the workaround in #166 for now - thanks @Be-P 👍

Same issue here! Hey @survived, any chance we can have this PR get merged and a new version get published?

shekohex avatar Aug 15 '22 16:08 shekohex

@elichai I've fixed the cargofmt step. The next step that runs cargo clippy -- -D clippy::all, throws errors from various files that were not touched by this PR. Should I go around fixing them?

Be-P avatar Aug 20 '22 21:08 Be-P

No need, I'll do that in a separate PR

elichai avatar Aug 22 '22 08:08 elichai

maintainers, don't you think the crate deserves a version bump to 0.9.1 or something? The problem addressed by this PR still exists for

curv-kzen = "0.9.0"
dsa = "0.4.0"

but does not for

curv-kzen = { git ="https://github.com/ZenGo-X/curv.git" }
dsa = "0.4.0"

vnermolaev avatar Sep 08 '22 11:09 vnermolaev