Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Generate keypair file for c program deployment #15080

Merged

Conversation

jackcmay
Copy link
Contributor

@jackcmay jackcmay commented Feb 4, 2021

Problem

No default keypair generated for c program deployment

Summary of Changes

generate one

Fixes #

@jackcmay jackcmay added the v1.5 label Feb 4, 2021
@jackcmay
Copy link
Contributor Author

jackcmay commented Feb 4, 2021

@mvines What do you think of calling solana-keygen in the C makefile? Are the solana tools (solana-keygen specifically) not available in the stable-perf ci build?

@mvines
Copy link
Contributor

mvines commented Feb 4, 2021

What do you think of calling solana-keygen in the C makefile?

I like the idea

Are the solana tools (solana-keygen specifically) not available in the stable-perf ci build?

Not in the PATH, I think we'd need to add a rule to build solana-keygen from source before using it in stable-perf

@jackcmay jackcmay force-pushed the generate-keypair-for-c-programs branch from 5d74eba to 45885b3 Compare February 4, 2021 19:09
@jackcmay jackcmay added noCI Suppress CI on this Pull Request CI Pull Request is ready to enter CI and removed noCI Suppress CI on this Pull Request CI Pull Request is ready to enter CI labels Feb 4, 2021
@solana-grimes solana-grimes removed the CI Pull Request is ready to enter CI label Feb 4, 2021
@jackcmay jackcmay merged commit bba1b49 into solana-labs:master Feb 4, 2021
mergify bot pushed a commit that referenced this pull request Feb 4, 2021
* Generate keypair file for c program deployment

* Build and use solana-keygen in test-stable-perf

(cherry picked from commit bba1b49)
mergify bot added a commit that referenced this pull request Feb 4, 2021
* Generate keypair file for c program deployment

* Build and use solana-keygen in test-stable-perf

(cherry picked from commit bba1b49)

Co-authored-by: Jack May <[email protected]>
@jackcmay jackcmay deleted the generate-keypair-for-c-programs branch April 7, 2021 20:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants