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.rebuild``
+     - Boolean, default ``true``. If ``false``, do not rebuild on west flash.
+
 Debugging: ``west debug``, ``west debugserver``
 ***********************************************
 
@@ -683,6 +699,25 @@ to debug::
 
 .. _west-runner:
 
+Configuration Options
+=====================
+
+You can :ref:`configure <west-config-cmd>` ``west debug`` and
+:ref:`configure <west-config-cmd>` ``west debugserver`` using these options.
+
+.. NOTE: docs authors: keep this table sorted alphabetically
+
+.. list-table::
+   :widths: 10 30
+   :header-rows: 1
+
+   * - Option
+     - Description
+   * - ``debug.rebuild``
+     - Boolean, default ``true``. If ``false``, do not rebuild on west debug.
+   * - ``debugserver.rebuild``
+     - Boolean, default ``true``. If ``false``, do not rebuild on west debugserver.
+
 Flash and debug runners
 ***********************
 

scripts/west_commands/run_common.py

@@ -142,8 +142,10 @@ def add_parser_common(command, parser_adder=None, parser=None):
                        help=argparse.SUPPRESS)
     group.add_argument('-r', '--runner',
                        help='override default runner from --build-dir')
-    group.add_argument('--skip-rebuild', action='store_true',
+    group.add_argument('-s', '--skip-rebuild', "--no-rebuild", action='store_true',
                        help='do not refresh cmake dependencies first')
+    group.add_argument('--force-rebuild', '--rebuild', action='store_true',
+                       help='force a refresh of the cmake dependencies before running')
     group.add_argument('--domain', action='append',
                        help='execute runner only for given domain')
 
@@ -244,8 +246,7 @@ def do_run_common(command, user_args, user_runner_args, domain_file=None):
             )
 
     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 +569,28 @@ 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):
+    rebuild_config = config.get(command.name, 'rebuild', fallback='true').lower()
+
+    # Truth table ->
+    # config rebuild | manual option | rebuild?
+    # ==========================================
+    #      X         |     skip      | skip
+    #      X         |     force     | rebuild
+    # true (default) |    [none]     | rebuild
+    # false          |    [none]     | skip
+    #============================================
+    # Therefore, only two cases in which we skip:
+
+    # user manually specified skip rebuild, so we skip
+    if args.skip_rebuild:
+        return
+
+    # config says to skip, and user did not manually override, so skip
+    if rebuild_config == 'false' and not args.force_rebuild:
+        return
+
+    # any other case, we do rebuild
+
     _banner(f'west {command.name}: rebuilding')
     try:
         zcmake.run_build(build_dir)
@@ -723,7 +746,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)