-
Notifications
You must be signed in to change notification settings - Fork 12
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
Update to zcash 5.2.0 #37
Conversation
git-subtree-dir: depend/zcash git-subtree-split: f98166f7f556ee9b9f597d45275c32843a87b0b6
77c2fb8
to
9d36477
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks ok, but the Windows build is failing.
Can we either fix it, or remove it and open a ticket?
b928948
to
0ccbbba
Compare
0ccbbba
to
aee0aee
Compare
I ended up removing Windows support and opening #38 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks good!
For ZcashFoundation/zebra#4767 we need to update zcash_script to use the latest zcash sources, since we must use the same dependencies as zcash to avoid dupicates.
In 5.1.0, zcash uses cxxbridge to generate bindings in build time, which means that we must do the same. This is not ideal since we must re-do the work that
zcash
already does when compiling, but the alternative would be to buildzcash
while building this crate which is even more awkward. The long-term solution would be to move this crate tozcash/zcash
, which we can consider later.