diff options
| author | Jack Rosenthal <jrosenth@chromium.org> | 2022-02-10 10:46:38 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2022-02-11 01:02:34 +0000 |
| commit | 41aeebed557b6dde239520910b96456675963c77 (patch) | |
| tree | 1a2c9f7d2ed525ab6ee4e370bff4dd18a25e647b /zephyr | |
| parent | 86bc4ab6082ccff0d9d63ca2364a37df0ffa859f (diff) | |
| download | chrome-ec-41aeebed557b6dde239520910b96456675963c77.tar.gz | |
zephyr: zmake: Remove the hard-coded list of projects in firmware_builder.py
This is actually just duplicate work of what the test phase is going
to end up doing, as testall will want to do the build anyway.
For now (that is, until we improve the zmake CLI enough to distinctly
separate the phases logically), let's just do nothing here.
BUG=b:217788621,b:218868887
BRANCH=none
TEST=CQ passes
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: I91cbdf45b80a4eec1ead03a8342790c476b36585
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3452933
Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
Diffstat (limited to 'zephyr')
| -rwxr-xr-x | zephyr/firmware_builder.py | 16 |
1 files changed, 3 insertions, 13 deletions
diff --git a/zephyr/firmware_builder.py b/zephyr/firmware_builder.py index f0ef0f7d96..ae19b82c81 100755 --- a/zephyr/firmware_builder.py +++ b/zephyr/firmware_builder.py @@ -31,19 +31,9 @@ def build(opts): with open(opts.metrics, 'w') as f: f.write(json_format.MessageToJson(metrics)) - targets = [ - 'projects/posix-ec', - 'projects/volteer/volteer', - ] - for target in targets: - print('Building {}'.format(target)) - cmd = ['zmake', '-D', 'configure', '-b'] - if opts.code_coverage: - cmd.append('--coverage') - cmd.append(target) - rv = subprocess.run(cmd, cwd=pathlib.Path(__file__).parent).returncode - if rv != 0: - return rv + # Nothing to do, as the test phase actually does the builds. + # TODO(b/217788621): Do a build-only here once we can separate build + # and test phases on zmake CLI. return 0 |