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

sdk: Rename ed25519-instructions -> ed25519-program #3632

Merged
merged 2 commits into from
Nov 14, 2024

Conversation

joncinque
Copy link

@joncinque joncinque commented Nov 14, 2024

Problem

The precompiles behave more like programs than instructions, as was decided with the new secp256r1 precompile in #3152, but the ed25519 precompile was split as ed25519-instructions in #3330.

Summary of changes

Rename the crate from solana-ed25519-instructions to solana-ed25519-program, which is a previous crate that used to be used until solana-labs#19930.

#### Problem

The precompiles behave more like programs than instructions, as was
decided with the new secp256r1 precompile in solana-labs#3152, but the ed25519
precompile was split as `ed25519-instructions`.

#### Summary of changes

Rename the crate from `solana-ed25519-instructions` to
`solana-ed25519-program`, which is a previous crate that used to be used
until solana-labs#19930.
Copy link

@febo febo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

kevinheavey added a commit to kevinheavey/agave that referenced this pull request Nov 14, 2024
kevinheavey added a commit to kevinheavey/agave that referenced this pull request Nov 14, 2024
@joncinque joncinque merged commit 8652ecc into anza-xyz:master Nov 14, 2024
52 checks passed
@joncinque joncinque deleted the mv-ed25519 branch November 14, 2024 16:37
kevinheavey added a commit to kevinheavey/agave that referenced this pull request Nov 14, 2024
kevinheavey added a commit to kevinheavey/agave that referenced this pull request Nov 14, 2024
kevinheavey added a commit to kevinheavey/agave that referenced this pull request Nov 15, 2024
kevinheavey added a commit to kevinheavey/agave that referenced this pull request Nov 16, 2024
kevinheavey added a commit to kevinheavey/agave that referenced this pull request Nov 20, 2024
kevinheavey added a commit to kevinheavey/agave that referenced this pull request Nov 20, 2024
febo pushed a commit that referenced this pull request Nov 20, 2024
* start extracting secp256k1-instruction

* fix imports and dependencies

* fix docs

* activate solana-secp256k1-instruction/bincode in solana-sdk

* import program id from solana_sdk_ids

* rename to solana-secp256k1-program like in #3632

* update lock file after rebase

* tighten deps

* minimise solana-sdk usage in docs and tests
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

Successfully merging this pull request may close these issues.

3 participants