-
Notifications
You must be signed in to change notification settings - Fork 272
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
Version numbers increase, even when building from same commit or tag #482
Comments
interesting discovery, I have to spin up a linux box to verify that. It seems to work differently maybe under osx or the code changed for the versioning (and I can't at the moment recall when we did that). I tested with 1.3.1 also and it also showed, what I expected. Example:
Then again. I downloaded the zip release, so maybe at a snapshot at that time it didn't have access to |
I'm adding this as a feature request. Your use case make sense and we should be able to update the version handling to avoid this. |
still testing it out but I believe this PR should fix it @phusen https://github.com/KjellKod/g3log/pull/515/files
|
resolved with #515
|
We are building g3log as a depenedency to another project. In our scripts, we clone the repo and checkout tag "1.3.3" in order to always build the same version. However, I noticed that this produces a
libg3logger.so
with increasing verision number over time, which was causing us a bit of trouble with our scripts. E.g. it just changed fromlibg3logger.so.1.3.2-210
tolibg3logger.so.1.3.2-211
. I think this is because the version number is generated usinggit rev-list --branches HEAD
When the
--brances
flag is included, the list includes commits from the "future" (i.e. which are not in the history of the current commit). So this means that the version will increase, when there are new commits in the repo, even though we are building the same thing. I'm really not an expert on generating version numbers using git, but this just seemed a bit counter-intuitive to me. Also, we can work around this for our use case - I just wanted to let you know about my findings.The text was updated successfully, but these errors were encountered: