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

dh-make-golang make: directly use the user preferred tag specified in "-git_revision" option #227

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wlyu2
Copy link
Contributor

@wlyu2 wlyu2 commented Nov 27, 2024

When a user specifies a valid tag when using

dh-make-golang make -git_revision {some valid tag}

if there is another tag pointing to the same commit as the specified tag, then dh-make-golang might use the other tag to determine the package version.

This patch modifies the behavior of "dh-make-golang make" such that, if the user specifies a valid tag in "-git_revision" option, "dh-make-golang make" will always use the specified tag to determine the upstream package version.

This patch also trivially changes the tests written in version_test.go so that the tests adjusts to the new function signature of version.go:pkgVersionFromGit().

@wlyu2
Copy link
Contributor Author

wlyu2 commented Nov 27, 2024

This patch attempts to fix #226 .

@ottok
Copy link
Contributor

ottok commented Jan 14, 2025

Does @n-peugnet or @Maytha8 want to review/approve this?

@wlyu2
Copy link
Contributor Author

wlyu2 commented Feb 11, 2025

Closes #226

… "-git_revision" option

When a user specifies a valid tag when using

    dh-make-golang make -git_revision {some valid tag}

if there is another tag pointing to the same commit as the specified
tag, then dh-make-golang might use the other tag to determine the
package version.

This patch modifies the behavior of "dh-make-golang make" such that, if
the user specifies a valid tag in "-git_revision" option,
"dh-make-golang make" will always use the specified tag to determine the
upstream package version.

This patch also trivially changes the tests written in version_test.go
so that the tests adjusts to the new function signature of
version.go:pkgVersionFromGit().

Signed-off-by: William Lyu <William.Lyu@windriver.com>
@wlyu2 wlyu2 force-pushed the fix-use-specified-tag branch from 4851485 to 4d16d16 Compare February 25, 2025 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants