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

Add base rust version to the output of cargo-build-sbf --version #32254

Merged
merged 1 commit into from
Jun 23, 2023
Merged

Add base rust version to the output of cargo-build-sbf --version #32254

merged 1 commit into from
Jun 23, 2023

Conversation

dmakarov
Copy link
Contributor

Problem

Platform tools version is not very informative. Additional information about the version of Rust the tools are based on could be helpful.

Summary of Changes

Add a function that extracts the version of the rust compiler installed in the platform-tools and output the version for the --version command line option of cargo-build-sbf.

sdk/cargo-build-sbf/src/main.rs Outdated Show resolved Hide resolved
sdk/cargo-build-sbf/src/main.rs Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Jun 23, 2023

Codecov Report

Merging #32254 (9ec2ec7) into master (9a620b4) will increase coverage by 0.0%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #32254   +/-   ##
=======================================
  Coverage    82.0%    82.0%           
=======================================
  Files         771      771           
  Lines      209337   209337           
=======================================
+ Hits       171740   171751   +11     
+ Misses      37597    37586   -11     

@dmakarov
Copy link
Contributor Author

Looks like it's ready to be merged.

@mvines mvines merged commit f71cf07 into solana-labs:master Jun 23, 2023
@mvines mvines added the v1.16 PRs that should be backported to v1.16 label Jun 23, 2023
mergify bot pushed a commit that referenced this pull request Jun 23, 2023
mvines pushed a commit that referenced this pull request Jun 24, 2023
…on (backport of #32254) (#32259)

Add base rust version to the output of cargo-build-sbf --version (#32254)

(cherry picked from commit f71cf07)

Co-authored-by: Dmitri Makarov <dmakarov@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
v1.16 PRs that should be backported to v1.16
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants