diff options
author | Randall Spangler <rspangler@chromium.org> | 2013-07-19 10:48:30 -0700 |
---|---|---|
committer | ChromeBot <chrome-bot@google.com> | 2013-07-19 18:01:55 -0700 |
commit | fe76b51da007bf51347ad3c3af549562a6d9979a (patch) | |
tree | e77abe8e897b96156ea0584bd2d9ad312138fb00 /util | |
parent | db347953e1a463abcb50669988b3f45bd7fbb826 (diff) | |
download | chrome-ec-fe76b51da007bf51347ad3c3af549562a6d9979a.tar.gz |
Truncate version string to 32 characters
The version struct and EC_CMD_GET_VERSION assume 32-character version
strings. But if the git tree is dirty and the board name is long, the
version string overflows that limit.
This change truncates what's stored in the version string to fit.
The build info still contains the full version string, as it did before.
BUG=chrome-os-partner:21156
BRANCH=none
TEST=build BOARD=mccroskey with a dirty tree; it should build.
Then cat build/mccroskey/ec_version.h to see CROS_EC_VERSION32 has
truncated the version string. Then build a platform of your choice
and type 'version' to see that the version string and build info is
still reported correctly.
Change-Id: Ie71b8efd99a83315f8b4d5ad10c51e48781b12f4
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/62649
Reviewed-by: Bill Richardson <wfrichar@chromium.org>
Diffstat (limited to 'util')
-rwxr-xr-x | util/getversion.sh | 21 |
1 files changed, 17 insertions, 4 deletions
diff --git a/util/getversion.sh b/util/getversion.sh index 06061ada93..4d1931d4a2 100755 --- a/util/getversion.sh +++ b/util/getversion.sh @@ -24,14 +24,27 @@ if ghash=`git rev-parse --short --verify HEAD 2>/dev/null`; then git status > /dev/null 2>&1 dirty=`sh -c "[ '$(git diff-index --name-only HEAD)' ] && echo '-dirty'"` - ver="${ver_major}.${ver_branch}.${numcommits}-${ghash}${dirty}" + vbase="${ver_major}.${ver_branch}.${numcommits}-${ghash}${dirty}" else - ver="no_version" + vbase="no_version" fi + +ver="${BOARD}_${vbase}" + +echo "/* This file is generated by util/getversion.sh */" + +echo "/* Version string for use by common/version.c */" echo "#ifdef SHIFT_CODE_FOR_TEST" -echo "#define VERSION \"${ver}_shift\"" +echo "#define CROS_EC_VERSION \"${ver}_shift\"" echo "#else" -echo "#define VERSION \"${ver}\"" +echo "#define CROS_EC_VERSION \"${ver}\"" echo "#endif" + +echo "/* Version string, truncated to 31 chars (+ terminating null = 32) */" +echo "#define CROS_EC_VERSION32 \"${ver:0:31}\"" + +echo "/* Sub-fields for use in Makefile.rules and to form build info string" +echo " * in common/version.c. */" +echo "#define VERSION \"${ver}\"" echo "#define DATE \"`date '+%F %T'`\"" echo "#define BUILDER \"${USER}@`hostname`\"" |