Merge pull request #880 from eyal0/dry_run

eyal0 · web-flow · commit e4a633de575e · 2026-04-03T20:37:23.000-06:00
feat: --dry-run argument to python tools.
diff --git a/tools/parallel.py b/tools/parallel.py
@@ -54,7 +54,7 @@ def main() -> None:
         help="Keep same order. If set, print outputs in input order; if not, print in completion order."
     )
     parser.add_argument(
-        "--dry-run",
+        "-n", "--dry-run",
         action="store_true",
         help="Print the command that would be run for each input instead of running it."
     )
diff --git a/tools/run_example.py b/tools/run_example.py
@@ -23,6 +23,11 @@ def parse_args() -> argparse.Namespace:
         required=True,
         help="Path to pcb2gcode binary to run.",
     )
+    parser.add_argument(
+        "-n", "--dry-run",
+        action="store_true",
+        help="Dry run (print the command that would be run instead of running it)",
+    )
     parser.add_argument(
         "example_dir",
         help="Example directory to run (e.g. a subdir of tests/data/gerbv_example)",
@@ -36,7 +41,7 @@ def main() -> None:
     code_coverage_enabled = args.code_coverage
     pcb2gcode_binary = args.pcb2gcode_binary
     pcb2gcode_binary = os.path.abspath(pcb2gcode_binary)
-
+    dry_run = args.dry_run
     have_valgrind = shutil.which("valgrind") is not None
     return_code = 0
     try:
@@ -75,9 +80,13 @@ def main() -> None:
             cmd = [pcb2gcode_binary]
 
         start = time.perf_counter()
-        result = subprocess.run(cmd, cwd=example_dir, stdout=subprocess.PIPE, stderr=subprocess.STDOUT, text=True)
-        print(result.stdout, end="")
-        return_code = result.returncode
+        if dry_run:
+            print(shlex.join(cmd))
+            return_code = 0
+        else:
+            result = subprocess.run(cmd, cwd=example_dir, stdout=subprocess.PIPE, stderr=subprocess.STDOUT, text=True)
+            print(result.stdout, end="")
+            return_code = result.returncode
         elapsed = time.perf_counter() - start
         print(f"real {elapsed:.3f} seconds")
         if return_code != 0:

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ def main() -> None:`
`54`	`54`	`help="Keep same order. If set, print outputs in input order; if not, print in completion order."`
`55`	`55`	`)`
`56`	`56`	`parser.add_argument(`
`57`		`- "--dry-run",`
	`57`	`+ "-n", "--dry-run",`
`58`	`58`	`action="store_true",`
`59`	`59`	`help="Print the command that would be run for each input instead of running it."`
`60`	`60`	`)`