commit 2e989bc69a6cbcaaf326094af8eb1c75ae882850
author Alex Klein <saklein@chromium.org> Tue May 02 14:19:25 2023 -0600
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue May 02 22:36:18 2023 +0000
tree 43e558aaf17649545de41974eee0a7e7400eb96d
parent f9fa452b3938716e25ca0748fbf14ca291d0a4b9

scripts: parallel_emerge: Minor cleanup.

line-too-long lint, % formatting to fstring literals.

BUG=None
TEST=CQ

Change-Id: I991579828020021599c07dd946430a624761682e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4500381
Reviewed-by: Sergey Frolov <sfrolov@google.com>
Tested-by: Alex Klein <saklein@chromium.org>
Commit-Queue: Alex Klein <saklein@chromium.org>
Auto-Submit: Alex Klein <saklein@chromium.org>
Commit-Queue: Sergey Frolov <sfrolov@google.com>

scripts/parallel_emerge.py

diff --git a/scripts/parallel_emerge.py b/scripts/parallel_emerge.py
index 908da30..e798c1f 100644
--- a/scripts/parallel_emerge.py
+++ b/scripts/parallel_emerge.py
@@ -39,8 +39,8 @@
     emerge.
 
     Returns:
-      commandline.ArgumentParser that captures arguments specific to
-      parallel_emerge
+        commandline.ArgumentParser that captures arguments specific to
+        parallel_emerge.
     """
     parser = commandline.ArgumentParser(description=__doc__, dryrun=True)
 
@@ -99,9 +99,9 @@
     parser.add_argument(
         "--eventlogfile",
         help=argparse.SUPPRESS,
-        deprecated="parallel_emerge no longer records failed packages. Set CROS_METRICS_DIR "
-        "in the system environment to get a log of failed packages and which "
-        "phase they failed in.",
+        deprecated="parallel_emerge no longer records failed packages. Set "
+        "CROS_METRICS_DIR in the system environment to get a log of failed "
+        "packages and which phase they failed in.",
     )
     parser.add_argument(
         "--show-output",
@@ -132,24 +132,23 @@
 
     if parsed_args.get("workon"):
         emerge_args.append(
-            "--reinstall-atoms=%s" % " ".join(parsed_args["workon"])
+            f"--reinstall-atoms={' '.join(parsed_args['workon'])}"
         )
         emerge_args.append(
-            "--usepkg-exclude=%s" % " ".join(parsed_args["workon"])
+            f"--usepkg-exclude={' '.join(parsed_args['workon'])}"
         )
 
     if parsed_args.get("force_remote_binary"):
         emerge_args.append(
-            "--useoldpkg-atoms=%s"
-            % " ".join(parsed_args["force_remote_binary"])
+            f"--useoldpkg-atoms={' '.join(parsed_args['force_remote_binary'])}"
         )
 
     if parsed_args.get("root_deps") is not None:
-        emerge_args.append("--root-deps=%s" % parsed_args["root_deps"])
+        emerge_args.append(f"--root-deps={parsed_args['root_deps']}")
     else:
         emerge_args.append("--root-deps")
 
-    emerge_args.append("--jobs=%s" % parsed_args["jobs"])
+    emerge_args.append(f"--jobs={parsed_args['jobs']}")
 
     # The -v/--verbose flag gets eaten by the commandline.ArgumentParser to
     # set the log_level, but it was almost certainly meant to be passed through