Complete platform tools cache by making temp crate a SBPF program
#220
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Building off #202
#202 downloads the platform tools, but
cargo build-sbfappears to run further setup steps when it tries to compile a SBPF program for the first time. These further setup steps involve downloading criterion and switching out the toolchain. Dump of stdout is pasted in appendix section below for visibility.This PR makes sure that those setup steps are ran as well by marking the
temp/repo as a SBPF program by settingin
Cargo.toml.Other
temp/folder bycd ../before removing itcargo build-sbftocargo test-sbfin case the test step performs any additional setup (doesn't look like it does)P.S. the bash command is getting a little unwieldy, idk if you guys want to factor it out into a bash script or something like that.
P.P.S the right thing to do would probably be to further figure out why the initial setup involves downloading a
criterionrelease. Seems wholly unnecessary.Appendix: setup stdout dump