From c2e00c29b74e13ecb26ef8345bef55293a342c65 Mon Sep 17 00:00:00 2001 From: Brian Paul Date: Wed, 14 Jun 2017 21:38:31 -0600 Subject: [PATCH] automake: increase the MESA_GIT_SHA1 hash id length from 7 to 10 digits The SCons build has been using 10 digits of the git hash id for the MESA_GIT_SHA1 string in git_sha1.h for about a year now. I bumped it up after running into a case where a 7-digit hash ID was ambiguous. This patch makes the same change for the autotools build. The command "git log | grep "^commit" | cut -b 8-14 | sort | uniq -d" shows there are currently 17 cases where 7 digits of hash id are ambiguous on master (probably quite a few more if we'd consider other branches). Instead of using "git log -n 1 --oneline" use "git rev-parse --short=10 HEAD" to get the HEAD hash id. v2: use printf instead of sed, per Eric's suggestion. Reviewed-by: Eric Engestrom --- src/Makefile.am | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Makefile.am b/src/Makefile.am index aa5f8aaf7d7..df912c442af 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -28,8 +28,8 @@ git_sha1.h.tmp: @touch git_sha1.h.tmp @if test -e $(top_srcdir)/.git; then \ if which git > /dev/null; then \ - git --git-dir=$(top_srcdir)/.git log -n 1 --oneline | \ - sed 's/^\([^ ]*\) .*/#define MESA_GIT_SHA1 "git-\1"/' \ + printf '#define MESA_GIT_SHA1 "git-%s"\n' \ + `git --git-dir=$(top_srcdir)/.git rev-parse --short=10 HEAD` \ > git_sha1.h.tmp ; \ fi \ fi -- 2.30.2