scripts: flash: Add west config for flash skip rebuild

Add a west config option to skip rebuilds by default or not when doing
west flash.

Signed-off-by: Declan Snyder <[email protected]>

Author: decsny

doc/develop/west/build-flash-debug.rst

@@ -551,6 +551,22 @@ only the image from this domain::
 
 .. _west-debugging:
 
+Configuration Options
+=====================
+
+You can :ref:`configure <west-config-cmd>` ``west flash`` using these options.
+
+.. NOTE: docs authors: keep this table sorted alphabetically
+
+.. list-table::
+   :widths: 10 30
+   :header-rows: 1
+
+   * - Option
+     - Description
+   * - ``flash.skip-rebuild``
+     - Boolean, default ``false``. If ``true``, do not rebuild on west flash.
+
 Debugging: ``west debug``, ``west debugserver``
 ***********************************************
 

scripts/west_commands/run_common.py

@@ -242,10 +242,10 @@ def do_run_common(command, user_args, user_runner_args, domain_file=None):
             import_from_path(
                 module.meta.get("name", "runners_ext"), Path(module.project) / runner["file"]
             )
+    build_dir = get_build_dir(user_args)
 
     build_dir = get_build_dir(user_args)
-    if not user_args.skip_rebuild:
-        rebuild(command, build_dir, user_args)
+    rebuild(command, build_dir, user_args)
 
     domains = get_domains_to_process(build_dir, user_args, domain_file)
 
@@ -568,6 +568,13 @@ def load_cmake_cache(build_dir, args):
         log.die(f'no CMake cache found (expected one at {cache_file})')
 
 def rebuild(command, build_dir, args):
+    # Check if skip-rebuild is configured globally
+    skip_rebuild_config = config.get('flash', 'skip-rebuild', fallback='false')
+    skip_rebuild = args.skip_rebuild or (skip_rebuild_config.lower() == 'true')
+
+    if skip_rebuild:
+        return
+
     _banner(f'west {command.name}: rebuilding')
     try:
         zcmake.run_build(build_dir)
@@ -723,7 +730,7 @@ def dump_context(command, args, unknown_args):
         get_all_domain = True
 
     # Re-build unless asked not to, to make sure the output is up to date.
-    if build_dir and not args.skip_rebuild:
+    if build_dir:
         rebuild(command, build_dir, args)
 
     domains = get_domains_to_process(build_dir, args, None, get_all_domain)