cmd: Add 'version' option #220
Conversation
This 'version' option prints the fakemachine version Signed-off-by: Dylan Aïssi <[email protected]>
|
CI fails because of #218 |
| @@ -11,6 +11,8 @@ import ( | |||
| "strings" | |||
| ) | |||
|
|
|||
| const Version = "0.0.10" | |||
There was a problem hiding this comment.
Hardcoding is somewhat likely to break unfortunately; I wonder if golang has some good ways to generate the version from e.g. git or a package release? Would also be good to include the git version if it's not a specific release
There was a problem hiding this comment.
Untested but pretty sure we can inject the version at build time with go build -ldflags "-X main.Version=$VERSION".
How do you feel about
- injecting the version in the GitHub action based on git commit
- injecting the same version in the Debian packaging
I looked for existing libraries to do this, like in rust, but couldn't find anything.
(Also the same will be true for debos)
There was a problem hiding this comment.
Is that what's typical in the go space (surely others do this as well)? But yeah in principle that sounds fine, my main concern is really that it's too easy to forget to update :)
This 'version' option prints the fakemachine version. Fix #200