Stefan Tauner has uploaded this change for review. ( https://review.coreboot.org/21827
Change subject: fixup! Convert flashrom to git ......................................................................
fixup! Convert flashrom to git
- make version string generation independent of the actual VCS used by not generating "unknown" in the makefile but letting getrevision do that - make hook installation independent of version string generation since they have nothing to do ith each other and there are no synergies anymore
Change-Id: Iedc9df4c033a70447b8b1b65c83764c769b02c3f --- M Makefile M util/getrevision.sh 2 files changed, 8 insertions(+), 11 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/27/21827/1
diff --git a/Makefile b/Makefile index 62a3869..fd1fc3b 100644 --- a/Makefile +++ b/Makefile @@ -528,17 +528,10 @@
# Makefile.VCS is used when packaging flashrom and is generated by the # export rule. If Makefile.VCS is not found, version info will be obtained -# using util/getrevision.sh or "unknown" if SCM metadata directory isn't found. +# using util/getrevision.sh. ifeq ($(wildcard Makefile.VCS),) -ifeq ($(wildcard .git),) -VERSION ?= unknown -MAN_DATE ?= unknown -else VERSION ?= $(shell ./util/getrevision.sh --local) -MAN_DATE ?= $(shell ./util/getrevision.sh -d $(PROGRAM).8.tmpl 2>/dev/null) -# This is also a convenient time to install hooks. -$(shell ./util/git-hooks/install.sh) -endif +MAN_DATE ?= $(shell ./util/getrevision.sh --date $(PROGRAM).8.tmpl 2>/dev/null) else include Makefile.VCS endif @@ -557,6 +550,9 @@ # Inform user of the version string $(info Replacing all version templates with $(VERSION).)
+# If a VCS is found then try to install hooks. +$(shell ./util/getrevision.sh -c 2>/dev/null && ./util/git-hooks/install.sh) + ############################################################################### # Default settings of CONFIG_* variables.
diff --git a/util/getrevision.sh b/util/getrevision.sh index e1580e5..0f97546 100755 --- a/util/getrevision.sh +++ b/util/getrevision.sh @@ -131,13 +131,14 @@
# Retrieve local revision info. local_revision() { + local r if git_is_file_tracked "$1" ; then - local r=$(git describe $(git_last_commit "$1")) + r=$(git describe $(git_last_commit "$1")) if git_has_local_changes "$1" ; then r="$r-dirty" fi else - return ${EXIT_FAILURE} + r="unknown" fi
echo "${r}"