Skip to content

bugfix: display version string correctly#732

Merged
lorenzo merged 1 commit intohadolint:masterfrom
m-ildefons:show-version
Nov 10, 2021
Merged

bugfix: display version string correctly#732
lorenzo merged 1 commit intohadolint:masterfrom
m-ildefons:show-version

Conversation

@m-ildefons
Copy link
Copy Markdown
Member

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
Copy Markdown
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