[LEDE-DEV] [PATCH] scripts/getver.sh: append short git hash based on upstream commit

Magnus Kroken mkroken at gmail.com
Sat Jan 14 13:34:01 PST 2017


The short git hash suffix printed by getver.sh is taken from the
latest local commit, change this to use the hash from latest
upstream commit if available. This is considered the intended
behavior based on commit message a642a11faca87e2a7bddc1fadb54253e2fc26e84,
introducing getver.sh.

Signed-off-by: Magnus Kroken <mkroken at gmail.com>
---
The referenced commit message says:
The new output format will look like "r2400+2-882472e" for dirty trees or like
"r2402-882472e" for clean ones.

Since the example hashes are the same, I take this to mean that this was the intended behavior.

Intended or not, I would suggest this anyway. If someone provides a LEDE version number from a dirty tree,
e.g. r2961+8-abcdef01, you need access to a LEDE Git tree and ability to run getver.sh to look up which
commit r2961 corresponds to. The short hash (abcdef01) is from the dirty tree, and can be anything that person has committed.
Appending the upstream base commit hash is useful when people ask for help and you want to track down the 
commit they're based on, and you may not have a running LEDE build environment available. Knowing that a tree
is dirty (+8) is useful as well, but the commit hash is only useful to people who have access to the dirty tree.

 scripts/getver.sh | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/scripts/getver.sh b/scripts/getver.sh
index ecf048f..4d594d3 100755
--- a/scripts/getver.sh
+++ b/scripts/getver.sh
@@ -40,7 +40,12 @@ try_git() {
 			REV="${UPSTREAM_REV}+$((REV - UPSTREAM_REV))"
 		fi
 
-		REV="${REV:+r$REV-$(git log --format="%h" -1)}"
+		if [ -n "$UPSTREAM_BASE" ]; then
+			REV="${REV:+r$REV-$(git log -n 1 --format="%h" $UPSTREAM_BASE)}"
+		else
+			REV="${REV:+r$REV-$(git log --format="%h" -1)}"
+		fi
+
 		;;
 	esac
 
-- 
2.1.4




More information about the Lede-dev mailing list