diff options
author | Jack Rosenthal <jrosenth@chromium.org> | 2021-09-21 00:25:28 +0000 |
---|---|---|
committer | Jack Rosenthal <jrosenth@chromium.org> | 2021-09-21 00:26:25 +0000 |
commit | 6606053ed3614cde874d0892fde82df02e6a5c94 (patch) | |
tree | a2554c4cde2a4a5be810af674bf2cf6b06569428 /zephyr/zmake/zmake | |
parent | 979c015f57d24b199611e4de1fe8ca2e4a12aafd (diff) | |
download | chrome-ec-6606053ed3614cde874d0892fde82df02e6a5c94.tar.gz |
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>
Diffstat (limited to 'zephyr/zmake/zmake')
-rw-r--r-- | zephyr/zmake/zmake/version.py | 45 | ||||
-rw-r--r-- | zephyr/zmake/zmake/zmake.py | 8 |
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", ) |