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

fix release tags not recognized when not on a branch #229

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

Conversation

wlyu2
Copy link

@wlyu2 wlyu2 commented Nov 28, 2024

"dh-make-golang" creates the "debianized" source repo from scratch. It creates an empty git repo, adds the upstream as a remote, and fetches the tags. However, before this patch, it is using "git fetch" with no additional flags or options. This results in upstream tags not on any branches not being picked up. This patch adds "--tags" flag to "git fetch" so that it can fetch all upstream tags.

"dh-make-golang" creates the "debianized" source repo from scratch. It
creates an empty git repo, adds the upstream as a remote, and fetches
the tags. However, before this patch, it is using "git fetch" with no
additional flags or options. This results in upstream tags not on any
branches not being picked up. This patch adds "--tags" flag to
"git fetch" so that it can fetch all upstream tags.

Signed-off-by: William Lyu <[email protected]>
@wlyu2
Copy link
Author

wlyu2 commented Nov 28, 2024

This merge request fixes #228 .

Copy link
Contributor

@n-peugnet n-peugnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can confirm that it indeed fixes the issue. @wlyu2: you can add the content of your second comment in your original post, this will close the issue automatically on merge.

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