-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Ship native programs in net/-based testnets and the solana docker image #1461
Conversation
c811deb
to
0ba6939
Compare
@jackcmay, any objections? |
Possibly, I have comments/questions posted to this pr |
I don't see any comments/questions on my end. Did you not hit the "Submit Review" button perhaps? |
SDK side: solana-labs/solana-web3.js#24 |
@@ -107,8 +107,6 @@ sys-info = "0.5.6" | |||
tokio = "0.1" | |||
tokio-codec = "0.1" | |||
untrusted = "0.6.2" | |||
|
|||
[dev-dependencies] |
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 was required in order to build those projects as part of the development tests. How are they built now?
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.
They build always
@@ -0,0 +1,28 @@ | |||
#!/bin/bash -e | |||
# | |||
# Installs native programs as |cargo install| doesn't know about them |
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.
crates.io does not include all projects in the workspace?
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.
No! Seems like they've punted on this entirely for now. cargo install
will only install programs
…#1461) chain parallel iterator Co-authored-by: HaoranYi <haoran.yi@solana.com>
No description provided.