git_describe() and --dirty about cmake-modules HOT 10 CLOSED

Would like to see this supported as well. Preferably it would also automatically re-run CMake if the repo changes from dirty to clean state and vice versa.

from cmake-modules.

Or is it because cmake is re-run only after a commit is done and that makes --dirty basically useless?

from cmake-modules.

Well, the script is set up to trigger a CMake re-run if a commit is done. Not sure about --dirty, haven't used it myself, though it does seem useful: is that a feature of only newer git versions? I could see changing the logic so that if we are just looking at head, we omit the commit and pass --dirty.

from cmake-modules.

No it's not new, I remember using git describe --tags --dirty for years.

from cmake-modules.

Hi, seems that this issue is still present. Maybe some more information here about this modification: https://stackoverflow.com/questions/16035240/why-is-git-describe-dirty-adding-a-dirty-suffix-when-describing-a-clean-ch

I'll try to fix this.

from cmake-modules.

The issue is still happening. If I prepare a patch just removing the hash from the call to git_describe would you merge it? I don't see a use-case for calling git-describe with anothe parameter than HEAD.

from cmake-modules.

Something like this works for my use-case:

0001-cmake-fix-git_describe-not-working-with-dirty.txt

Edit: this does NOT work. Removing the hash from the list of parameters has the side-effect that cmake does not re-generate the version string when the git commit changes.

from cmake-modules.

I also recommend removing "ERROR_QUIET". Failing the build with the git error visible in the logs is better than silently printing "128-NOTFOUND" as a version string in my opinion.

from cmake-modules.

agreed on ERROR_QUIET

from cmake-modules.

There is now a git_describe_working_tree that does a git describe --dirty without the fancy commit-ish magic of the normal function. I think this should solve things.

from cmake-modules.