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

Proposed penumbra-crypto code reorganization #2765

Closed
19 tasks done
redshiftzero opened this issue Jun 29, 2023 · 1 comment · Fixed by #2798
Closed
19 tasks done

Proposed penumbra-crypto code reorganization #2765

redshiftzero opened this issue Jun 29, 2023 · 1 comment · Fixed by #2798
Assignees

Comments

@redshiftzero
Copy link
Member

redshiftzero commented Jun 29, 2023

This is a more detailed proposal for the final item in #2288. As described there, penumbra-crypto has become a catch-all crate for a bunch of diverse functionality. The scope of this ticket is to:

  1. Rehome the various modules of penumbra-crypto to existing crates that make sense, or create new crates if needed,
  2. Remove code in penumbra-crypto we're not using, i.e. stuff that was originally targeted for Penumbra mainnet but will end up going into PenumbraV2, possibly with significant changes.

Concrete steps

Note that the following paths refer to the various modules in penumbra-crypto in crates/core/crypto.

hdevalence added a commit that referenced this issue Jul 1, 2023
hdevalence added a commit that referenced this issue Jul 1, 2023
redshiftzero added a commit that referenced this issue Jul 2, 2023
redshiftzero added a commit that referenced this issue Jul 2, 2023
redshiftzero added a commit that referenced this issue Jul 2, 2023
redshiftzero added a commit that referenced this issue Jul 2, 2023
@redshiftzero redshiftzero self-assigned this Jul 6, 2023
@conorsch
Copy link
Contributor

conorsch commented Jul 7, 2023

Ticket still open to track cleanup, but we've largely achieved the reorg we wanted in advance of pending security audits. 🎉

conorsch added a commit to penumbra-zone/galileo that referenced this issue Jul 11, 2023
We abolished the crypto crate as part of Testnet 56, so we must update
our imports accordingly. Matches changes described in
penumbra-zone/penumbra#2765
conorsch added a commit to penumbra-zone/galileo that referenced this issue Jul 11, 2023
We abolished the crypto crate as part of Testnet 56, so we must update
our imports accordingly. Matches changes described in
penumbra-zone/penumbra#2765
conorsch added a commit to penumbra-zone/osiris that referenced this issue Jul 11, 2023
We abolished the crypto crate as part of Testnet 56, so we must update
our imports accordingly. Matches changes described in
penumbra-zone/penumbra#2765
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Testnet 56: Callisto
Development

Successfully merging a pull request may close this issue.

2 participants