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.

Sign up for GitHub

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

Jump to bottom

Stop using zcash_primitives/src/sapling/redjubjub.rs in favour of redjubjub crate #581

Closed
daira opened this issue Jul 4, 2022 · 1 comment
Closed

Stop using zcash_primitives/src/sapling/redjubjub.rs in favour of redjubjub crate #581

daira opened this issue Jul 4, 2022 · 1 comment
Labels
C-cleanup
Milestone
Extract sapling-c...

Comments

@daira
Copy link
Contributor

daira commented Jul 4, 2022

From zcashd v5.1.0, zcash_primitives/src/sapling/redjubjub.rs is used for RedJubjub signing, and the redjubjub crate for (batch) verification. Change to using the latter for both, to avoid duplication and reduce maintenance cost.
@daira daira added the C-cleanup label Jul 4, 2022
@str4d str4d mentioned this issue Nov 30, 2023
Closed
@str4d str4d added this to the Extract sapling-crypto milestone Nov 30, 2023
@daira
Copy link
Contributor Author

daira commented Dec 19, 2023

This was fixed by #1056

@daira daira closed this as completed Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-cleanup
Projects
None yet
Milestone
Extract sapling-crypto
Development

No branches or pull requests
2 participants
@daira @str4d