Skip to content
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

Add version command to print the cargo version. #487

Merged
merged 6 commits into from
Apr 28, 2022

Conversation

mcdee
Copy link
Contributor

@mcdee mcdee commented Feb 23, 2022

This adds a command tmkms version which simply prints out the version number as per the cargo information (or "unknown" if that is not available). It is useful for tools that want to check which version of the code is running.

(The version is available in the logs from startup, but logs rotate so this is often practically unavailable.)

@tony-iqlusion
Copy link
Member

@mcdee can you please rebase your branch? Thanks!

@mcdee
Copy link
Contributor Author

mcdee commented Apr 28, 2022

Done.

@tony-iqlusion
Copy link
Member

@mcdee needs rustfmt (and another rebase as I've merged another PR)

@mcdee
Copy link
Contributor Author

mcdee commented Apr 28, 2022

Updated.

use std::{ffi::OsStr, str};

#[test]
fn test_versionx() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo?

Copy link
Member

@tony-iqlusion tony-iqlusion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One apparent typo but otherwise looks good

@tarcieri tarcieri merged commit f3a7fc9 into iqlusioninc:main Apr 28, 2022
@tony-iqlusion tony-iqlusion mentioned this pull request May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants