diff options
author | Kevin Shelton <kmshelton@chromium.org> | 2021-02-01 17:47:21 +0000 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-02-01 19:39:41 +0000 |
commit | f1e647139e6a7f0a2620116c026542a9f56278bf (patch) | |
tree | 43246bbe6e567ebff9c99224ba442ee3e31c6979 | |
parent | 4344a16290c13b0ddbab4656774a86058ea7ee48 (diff) | |
download | chrome-ec-f1e647139e6a7f0a2620116c026542a9f56278bf.tar.gz |
firmware_builder: style cleanup
Apply style conformance changes in preparation for logic changes
(formatting changes programmatically applied via yapf).
BUG=b:176926834
TEST=None
BRANCH=None
Change-Id: If5236c068bbefa765c47c81a8580d35a89aef6d9
Signed-off-by: Kevin Shelton <kmshelton@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2664766
Commit-Queue: LaMont Jones <lamontjones@chromium.org>
Reviewed-by: LaMont Jones <lamontjones@chromium.org>
-rwxr-xr-x | firmware_builder.py | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/firmware_builder.py b/firmware_builder.py index 428657158b..940412af06 100755 --- a/firmware_builder.py +++ b/firmware_builder.py @@ -18,6 +18,7 @@ from google.protobuf import json_format from chromite.api.gen.chromite.api import firmware_pb2 + def build(opts): """Builds all EC firmware targets""" # TODO(b/169178847): Add appropriate metric information @@ -25,7 +26,8 @@ def build(opts): with open(opts.metrics, 'w') as f: f.write(json_format.MessageToJson(metrics)) subprocess.run(['make', 'buildall_only', '-j{}'.format(opts.cpus)], - cwd=os.path.dirname(__file__), check=True) + cwd=os.path.dirname(__file__), + check=True) def test(opts): @@ -37,18 +39,20 @@ def test(opts): # Verify all posix-based unit tests build and pass subprocess.run(['make', 'runtests', '-j{}'.format(opts.cpus)], - cwd=os.path.dirname(__file__), check=True) + cwd=os.path.dirname(__file__), + check=True) # Verify compilation of the on-device unit test binaries. # TODO(b/172501728) These should build for all boards, but they've bit # rotted, so we only build the ones that compile. subprocess.run( ['make', 'BOARD=bloonchipper', 'tests', '-j{}'.format(opts.cpus)], - cwd=os.path.dirname(__file__), check=True) + cwd=os.path.dirname(__file__), + check=True) def main(args): - """Builds and tests all of the EC targets and reports build metrics""" + """Builds and tests all of the EC targets and reports build metrics.""" opts = parse_args(args) if not hasattr(opts, 'func'): |