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

Failing to build because of missing dependency version #112

Closed
alexbouez opened this issue Apr 22, 2021 · 10 comments
Closed

Failing to build because of missing dependency version #112

alexbouez opened this issue Apr 22, 2021 · 10 comments

Comments

@alexbouez
Copy link

I am unable to build with cargo because of a missing dependency "zeroize 0.10" which has been removed,
Newer versions are available (1.3.0, 1.2.0, 1.1.1).

@omershlo
Copy link
Contributor

Thanks @alexbouez !
We are aware and @survived is working on a fix. We will update shortly !

@omershlo
Copy link
Contributor

Hi, fix is taking some time,
Please try to work on one of the earlier tags instead of latest.

@alexbouez
Copy link
Author

Hello, thank you for your help,

A quick fix that works for me is downloading a local version of zeroize (for example from crates),
changing its version tag to "0.10.1"
and adding a line to the Patch manifeset in Cargo.toml:
zeroize = { version = "0.10.1", path = "zeroize" }

I am running cargo version 1.53.0-nightly on VSCode.
Current repo commit a072793725f279f467d0e9f81480afc27cf5f17a

@survived
Copy link
Contributor

Hi @alexbouez, please check if #113 solves the issue (try tag v0.6.2)

@vherbert
Copy link

vherbert commented May 11, 2021

Hello!

I meet the same build error, but I use feature ["ec_ed25519"] and tag v0.6.2 does not have this feature.

Thanks for your help.

@survived
Copy link
Contributor

Hi @vherbert, all features were removed from curv in past versions. We used to choose EC point implementation by specifying a feature, now all curve implementation are available in curv::elliptic::curves::*, and all structures and algorithms are generic over choice of curve (ECPoint trait). I.e. curv API has been changed and some manual actions are required to move on next version.

@vherbert
Copy link

Ok, thx. It is maybe not the good place to say it but this old feature is still employed in current version of multi-party-eddsa.

@omershlo
Copy link
Contributor

thanks @vherbert - It would be great if you could update multi-party-eddsa repo?
otherwise - please open an issue

@survived
Copy link
Contributor

survived commented May 12, 2021

Right, multi-party-eddsa has not been updated yet

@survived
Copy link
Contributor

I believe this issue should be closed - zeroize dependency updated

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

No branches or pull requests

4 participants