Skip to content

[fix] Include git commit hash in tarballs #3124

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

Merged
merged 3 commits into from
Apr 19, 2025

Conversation

dominikschulz
Copy link
Member

This change should allow builds made from a release tarball to accurately display the git commit hash it was built from.

It does so by rendering the current short hash when packaging the tarball. This workaround is necessary since the tarball won't include the .git directory so we can't extract the commit information from it.

This change should allow builds made from a release tarball to
accurately display the git commit hash it was built from.

It does so by rendering the current short hash when packaging
the tarball. This workaround is necessary since the tarball
won't include the .git directory so we can't extract the
commit information from it.

Signed-off-by: Dominik Schulz <[email protected]>
Signed-off-by: Dominik Schulz <[email protected]>
Signed-off-by: Dominik Schulz <[email protected]>
@dominikschulz dominikschulz merged commit f5c694c into gopasspw:master Apr 19, 2025
9 checks passed
@dominikschulz dominikschulz deleted the fix/issue-1336 branch April 19, 2025 12:45
sudoforge pushed a commit to sudoforge/gopass that referenced this pull request May 20, 2025
* [fix] Include git commit hash in tarballs

This change should allow builds made from a release tarball to
accurately display the git commit hash it was built from.

It does so by rendering the current short hash when packaging
the tarball. This workaround is necessary since the tarball
won't include the .git directory so we can't extract the
commit information from it.

Signed-off-by: Dominik Schulz <[email protected]>

* [fix] Fix TestVersionPrinter

Signed-off-by: Dominik Schulz <[email protected]>

* [fix] Fix spacing

Signed-off-by: Dominik Schulz <[email protected]>

---------

Signed-off-by: Dominik Schulz <[email protected]>
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.

1 participant