Skip to content

fix: Update version reporting logic #1244

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 1 commit into from
Mar 12, 2025
Merged

Conversation

mem
Copy link
Contributor

@mem mem commented Mar 7, 2025

Go 1.24 added a way to obtain the version (extract from the current Git repo) from the running program. Switch to that instead of passing linker flags at build time.

@mem mem requested a review from a team as a code owner March 7, 2025 23:53
@mem mem requested review from ka3de and The-9880 March 7, 2025 23:53
@mem mem force-pushed the mem/update-version-reporting-logic branch from f77b3f4 to 2d839f8 Compare March 7, 2025 23:54
@mem mem force-pushed the mem/update-version-reporting-logic branch from 2d839f8 to c765218 Compare March 10, 2025 20:44
Go 1.24 added a way to obtain the version (extract from the current Git
repo) from the running program. Switch to that instead of passing
linker flags at build time.

Update ./scripts/version to report the same string.

Signed-off-by: Marcelo E. Magallon <[email protected]>
@mem mem force-pushed the mem/update-version-reporting-logic branch from c765218 to 14421d4 Compare March 11, 2025 21:06
@mem mem merged commit ad3ffbb into main Mar 12, 2025
5 checks passed
@mem mem deleted the mem/update-version-reporting-logic branch March 12, 2025 15:32
@sm-release-app sm-release-app bot mentioned this pull request Mar 12, 2025
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 participants