dist/tools/ci: include rust, node and git commit in print-version#21734
dist/tools/ci: include rust, node and git commit in print-version#21734crasbe merged 2 commits intoRIOT-OS:masterfrom
Conversation
|
Looks good, I just asked the other maintainers with more Rust experience if anything else should be evaluated too (since I don't know much about that kind of Rust 😅 ) |
|
While rustup is not technically required, we check through it whether the right cross toolchain is present, for it is the only widespread way to install those. (Debian ships cargo, but no cross toolchains.) |
Added 😄 I think on arch there is also a way to install Rust without rustup so its probably a common mistake people dont really notice (esp. in their first steps with Rust) |
|
I just noticed on my work pc that riscv32-unknown-elf is not tracked, partly because the riscv toolchain ecosystem is confusing as heck and everyone has their own names. riscv32-unknown-elf is used on Arch though. (btw I use Arch) The output looks like this now: |
|
Hoorrraaaay 🥳 |
48a7325 to
37e4417
Compare
Contribution description
Currently the print-version script used for debugging does not include information about the Rust and Frontend environment. While at it I thought it might also make sense to include information about the exact state the RIOT repository is in, esp. to debug cases where the RIOT repository might simply be outdated.
Before:
After:
Testing procedure
make print-versionIssues/PRs references