Revert "zmake: Trim short version on full hashes"

This reverts commit 695b3c909c4846493eacd324e9c26933847372d1.

Reason for revert: Broke CQ

See https://ci.chromium.org/ui/p/chromeos/builders/postsubmit/trogdor-zephyr-postsubmit/314

Original change's description:
> zmake: Trim short version on full hashes
>
> The ec_version.h contains both a long build string and also a 31 char
> string. Change the short string to omit the commit count, and only
> include full hashes, i.e. hayato_v2.6-ec:5bd1aa,os:ade7b4 not
> hayato_v2.6.73670-ec:5bd1aa,os:
>
> Change test to be a real version string with hashes of the right length.
>
> BUG=None
> BRANCH=None
> TEST=./zephyr/zmake/run_tests.sh
>
> Change-Id: I18d73345036933594b8c74187796c3727f868629
> Signed-off-by: Jeremy Bettis <jbettis@google.com>
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3160464
> Tested-by: Jeremy Bettis <jbettis@chromium.org>
> Auto-Submit: Jeremy Bettis <jbettis@chromium.org>
> Commit-Queue: Jack Rosenthal <jrosenth@chromium.org>
> Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>

Bug: None
Change-Id: Ib4282f761c514c0ca2da6249d38ccc9cba852f6d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3172040
Auto-Submit: Jack Rosenthal <jrosenth@chromium.org>
Tested-by: Jack Rosenthal <jrosenth@chromium.org>
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>

2 files changed, 17 insertions, 36 deletions

diff --git a/zephyr/zmake/zmake/version.py b/zephyr/zmake/zmake/version.py
index c5506e9921..47aba6d804 100644
--- a/zephyr/zmake/zmake/version.py
+++ b/zephyr/zmake/zmake/version.py
@@ -86,14 +86,12 @@ def get_version_string(project, zephyr_base, modules, static=False):
             commits.
 
     Returns:
-        A tuple of version string which can be placed in FRID, FWID, or used in
-        the build for the OS, and a shortened string which is 31 chars or less.
+        A version string which can be placed in FRID, FWID, or used in
+        the build for the OS.
     """
     major_version, minor_version, *_ = util.read_zephyr_version(zephyr_base)
     project_id = project.project_dir.parts[-1]
     num_commits = 0
-    # Omit num_commits from short version string.
-    short_version_string = "{}_v{}.{}".format(project_id, major_version, minor_version)
 
     if static:
         vcs_hashes = "STATIC"
@@ -106,38 +104,23 @@ def get_version_string(project, zephyr_base, modules, static=False):
         for repo in repos.values():
             num_commits += _get_num_commits(repo)
 
-        vcs_parts = []
-        for name, repo in sorted(
-            repos.items(),
-            # Put the EC module first, then Zephyr OS kernel, as
-            # these are probably the most important hashes to
-            # developers.
-            key=lambda p: (p[0] != "ec", p[0] != "os", p),
-        ):
-            # Add hashes to vcs_hashes and short_version_string if it is still
-            # short enough.
-            new_short_ver = short_version_string
-            val = "{}:{}".format(name, _get_revision(repo)[:6])
-            vcs_parts.append(val)
-            vcs_hashes = ",".join(vcs_parts)
-            new_short_ver = "{}_v{}.{}-{}".format(
-                project_id, major_version, minor_version, vcs_hashes
+        vcs_hashes = ",".join(
+            "{}:{}".format(name, _get_revision(repo)[:6])
+            for name, repo in sorted(
+                repos.items(),
+                # Put the EC module first, then Zephyr OS kernel, as
+                # these are probably the most important hashes to
+                # developers.
+                key=lambda p: (p[0] != "ec", p[0] != "os", p),
             )
+        )
 
-            if len(new_short_ver) <= 31:
-                short_version_string = new_short_ver
-
-    long_version_string = "{}_v{}.{}.{}-{}".format(
+    return "{}_v{}.{}.{}-{}".format(
         project_id, major_version, minor_version, num_commits, vcs_hashes
     )
-    # If the long version string is short enough use it.
-    if len(long_version_string) <= 31:
-        short_version_string = long_version_string
-    assert len(short_version_string) <= 31
-    return long_version_string, short_version_string
 
 
-def write_version_header(version_str, short_version_str, output_path, static=False):
+def write_version_header(version_str, output_path, static=False):
     """Generate a version header and write it to the specified path.
 
     Generate a version header in the format expected by the EC build
@@ -165,7 +148,7 @@ def write_version_header(version_str, short_version_str, output_path, static=Fal
         output.write("#define {} {}\n".format(name, value))
 
     add_def("VERSION", version_str)
-    add_def("CROS_EC_VERSION32", short_version_str)
+    add_def("CROS_EC_VERSION32", version_str[:31])
 
     if static:
         add_def("BUILDER", "reproducible@build")
diff --git a/zephyr/zmake/zmake/zmake.py b/zephyr/zmake/zmake/zmake.py
index 34e25cc365..1b60e66f66 100644
--- a/zephyr/zmake/zmake/zmake.py
+++ b/zephyr/zmake/zmake/zmake.py
@@ -376,7 +376,7 @@ class Zmake:
         project = zmake.project.Project(build_dir / "project")
 
         # Compute the version string.
-        version_string, short_version_string = zmake.version.get_version_string(
+        version_string = zmake.version.get_version_string(
             project,
             build_dir / "zephyr_base",
             zmake.modules.locate_from_directory(build_dir / "modules"),
@@ -387,7 +387,6 @@ class Zmake:
         # configure was run.
         zmake.version.write_version_header(
             version_string,
-            short_version_string,
             build_dir / "include" / "ec_version.h",
         )
 
@@ -443,7 +442,7 @@ class Zmake:
         if output_files_out is None:
             output_files_out = []
         for output_file, output_name in project.packer.pack_firmware(
-            packer_work_dir, self.jobserver, version_string=short_version_string, **dirs
+            packer_work_dir, self.jobserver, version_string=version_string, **dirs
         ):
             shutil.copy2(output_file, output_dir / output_name)
             self.logger.debug("Output file '%s' created.", output_file)
@@ -592,7 +591,7 @@ class Zmake:
             return rv
 
         # Compute the version string.
-        version_string, short_version_string = zmake.version.get_version_string(
+        version_string = zmake.version.get_version_string(
             project,
             build_dir / "zephyr_base",
             zmake.modules.locate_from_directory(build_dir / "modules"),
@@ -603,7 +602,6 @@ class Zmake:
         # configure was run.
         zmake.version.write_version_header(
             version_string,
-            short_version_string,
             build_dir / "include" / "ec_version.h",
         )