这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@heinezen
Copy link
Member

Fixes incorrect version number being displayed in source builds.

@heinezen heinezen added area: buildsystem Related to our cmake/python buildsystem bugfix Restores intended behavior labels Oct 10, 2023
Copy link
Member

@TheJJ TheJJ left a comment

Choose a reason for hiding this comment

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

https://mirrors.edge.kernel.org/pub/software/scm/git/docs/git-describe.html

--tags 

Instead of using only the annotated tags, use any tag found in refs/tags namespace. This option enables matching a lightweight (non-annotated) tag. 

so it used non-annonated tags before, now only annotated tags?

@heinezen
Copy link
Member Author

so it used non-annonated tags before, now only annotated tags?

No it's the other way around

Before this PR: Only annotated tags were used (resulting in 0.4.1 being used as the version number by the build system since this is the last annotated PR)
With this PR: All tags (annotated or not) are considered, i.e. the correct version 0.5.1 should be used

@heinezen heinezen force-pushed the fix/version-display branch from 92fd04e to 20e9b94 Compare October 12, 2023 16:24
@TheJJ TheJJ merged commit db164ef into SFTtech:master Oct 15, 2023
@heinezen heinezen deleted the fix/version-display branch October 20, 2023 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: buildsystem Related to our cmake/python buildsystem bugfix Restores intended behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants