commit 57d81cd8ebe8f3667a42e797d6bedeb5d223718a
author Mike Frysinger <vapier@chromium.org> Thu Feb 02 09:05:06 2023 -0500
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Feb 08 06:08:41 2023 +0000
tree 943f9adb1234511f0f29b378f1b91a7131ef7330
parent 61b792cabb41095958b417d61b9902163eff0574

scripts: parallel_emerge: convert to common --dry-run options

BUG=None
TEST=CQ passes

Change-Id: I11292a372b19d6644dcadd72b1e203e548fc378d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4219581
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Lizzy Presland <zland@google.com>
Tested-by: Mike Frysinger <vapier@chromium.org>

scripts/parallel_emerge.py

diff --git a/scripts/parallel_emerge.py b/scripts/parallel_emerge.py
index ca92421..8a0320b 100644
--- a/scripts/parallel_emerge.py
+++ b/scripts/parallel_emerge.py
@@ -42,7 +42,7 @@
       commandline.ArgumentParser that captures arguments specific to
       parallel_emerge
     """
-    parser = commandline.ArgumentParser()
+    parser = commandline.ArgumentParser(description=__doc__, dryrun=True)
 
     board_group = parser.add_mutually_exclusive_group()
     board_group.add_argument(
@@ -90,16 +90,6 @@
         default=multiprocessing.cpu_count(),
         metavar="PARALLEL_JOBCOUNT",
     )
-    parser.add_argument(
-        "-n",
-        "--dryrun",
-        "--dry-run",
-        dest="dry_run",
-        action="store_true",
-        default=False,
-        help="Print the emerge command that would have been run instead of "
-        "actually running it.",
-    )
 
     parser.add_argument(
         "--retries",
@@ -175,7 +165,7 @@
 
     cmd = ["emerge"] + emerge_args
     cmd_str = cros_build_lib.CmdToStr(cmd)
-    if parsed_args.get("dry_run"):
+    if parsed_args.get("dryrun"):
         logging.notice("Would have run: %s", cmd_str)
         return