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

Compilation error when using grin_wallet_libwallet as dependency #556

Closed
jafalter opened this issue Dec 28, 2020 · 2 comments
Closed

Compilation error when using grin_wallet_libwallet as dependency #556

jafalter opened this issue Dec 28, 2020 · 2 comments

Comments

@jafalter
Copy link
Contributor

jafalter commented Dec 28, 2020

Describe the bug
When adding grin_wallet_libwallet as a dependency to a Rust project like this:
image`
I receive a compilation error which seems to be coming from an issue with a dalek and the grin-core dependency. The bug appears for any version of grin_wallet_libwallet above 3.0.0. (2.1.0 and below works fine)

error: aborting due to 26 previous errors

Some errors have detailed explanations: E0026, E0027, E0308, E0560, E0599, E0609, E0615.
For more information about an error, try `rustc --explain E0026`.
error: could not compile `grin_wallet_libwallet`.

To Reproduce
Steps to reproduce the behavior:

  1. mkdir minimal
  2. cd minimal
  3. cargo init
  4. Add grin_wallet_libwallet = "4.0.0" as dependency to Cargo.toml
  5. cargo build

Expected behavior
Should compile without errors

Desktop (please complete the following information):

  • Ubuntu 20.04
  • Rust 1.47.0

Screenshots
image
image
image
image
image
image
image

@trevyn
Copy link
Contributor

trevyn commented Mar 16, 2021

This appears to be fixed in grin_wallet_libwallet = 5.0.1.

I can reproduce the issue with Rust 1.50 and grin_wallet_libwallet = 4.0.0, but this works for me:

[dependencies]
grin_wallet_libwallet = "5.0.1"
serde = "=1.0.118"

(Note that serde = "=1.0.118" is required due to serde-rs/serde#1949. This is fixed in mimblewimble/grin#3585, but that is not yet released to crates.io.)

@quentinlesceller
Copy link
Member

Can confirm @trevyn message. Thanks! Closing.

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

3 participants