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

bugfix: display version string correctly #732

Merged
merged 1 commit into from Nov 10, 2021

Conversation

m-ildefons
Copy link
Collaborator

Construct the version string correctly when not compiling in a git
directory.
This bug was caused by a typo and a misunderstanding of the code when
the version string logic was moved from app/Main.hs to
src/Hadolint/Meta.hs in commit 3b1de5a

fixes: #730

How to verify it

To verify that this actually fixes the issue, check out this commit, remove or rename the .git directory and compile Hadolint from source. The result should show the version string from the .cabal file and the -no-git ending.

$ hadolint -v
Haskell Dockerfile Linter 2.8.0-no-git

When compiling from source with the .git directory, the result should show the corresponding git tag and the short git hash.

Construct the version string correctly when not compiling in a git
directory.
This bug was caused by a typo and a misunderstanding of the code when
the version string logic was moved from app/Main.hs to
src/Hadolint/Meta.hs in commit 3b1de5a

fixes: hadolint#730
@lorenzo lorenzo merged commit 0b25353 into hadolint:master Nov 10, 2021
@lorenzo
Copy link
Member

lorenzo commented Nov 10, 2021

thanks for fixing it

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.

[2.8.0] pre-built binary releases for Linux and Mac report version as UNKNOWN
2 participants