sanitycheck: disable erroring on warnings

--warnings-as-errors was basically doing nothing, it is the default.
Replace this with an option to disable erroring on warning:

 -W, --disable-warnings-as-errors

Fixes zephyrproject-rtos#26910

Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Author: nashif

scripts/sanity_chk/sanitylib.py

@@ -1764,17 +1764,21 @@ def run_build(self, args=[]):
 
     def run_cmake(self, args=[]):
 
-        ldflags = "-Wl,--fatal-warnings"
-        logger.debug("Running cmake on %s for %s" % (self.source_dir, self.platform.name))
+        if self.warnings_as_errors:
+            ldflags = "-Wl,--fatal-warnings"
+            cflags = "-Werror"
+            aflags = "-Wa,--fatal-warnings"
+        else:
+            ldflags = cflags = aflags = ""
 
-        # fixme: add additional cflags based on options
+        logger.debug("Running cmake on %s for %s" % (self.source_dir, self.platform.name))
         cmake_args = [
-            '-B{}'.format(self.build_dir),
-            '-S{}'.format(self.source_dir),
-            '-DEXTRA_CFLAGS="-Werror ',
-            '-DEXTRA_AFLAGS=-Wa,--fatal-warnings',
-            '-DEXTRA_LDFLAGS="{}'.format(ldflags),
-            '-G{}'.format(self.generator)
+            f'-B{self.build_dir}',
+            f'-S{self.source_dir}',
+            f'-DEXTRA_CFLAGS="{cflags}"',
+            f'-DEXTRA_AFLAGS="{aflags}',
+            f'-DEXTRA_LDFLAGS="{ldflags}"',
+            f'-G{self.generator}'
         ]
 
         if self.cmake_only:
@@ -1915,6 +1919,7 @@ def __init__(self, suite, instance, **kwargs):
         self.generator = kwargs.get('generator', None)
         self.generator_cmd = kwargs.get('generator_cmd', None)
         self.verbose = kwargs.get('verbose', None)
+        self.warnings_as_errors = kwargs.get('warnings_as_errors', True)
 
     @staticmethod
     def log_info(filename, inline_logs):
@@ -2298,6 +2303,7 @@ def __init__(self, board_root_list=[], testcase_roots=[], outdir=None):
         self.west_runner = None
         self.generator = None
         self.generator_cmd = None
+        self.warnings_as_errors = True
 
         # Keep track of which test cases we've filtered out and why
         self.testcases = {}
@@ -2903,7 +2909,8 @@ def calc_one_elf_size(instance):
                                         inline_logs=self.inline_logs,
                                         generator=self.generator,
                                         generator_cmd=self.generator_cmd,
-                                        verbose=self.verbose
+                                        verbose=self.verbose,
+                                        warnings_as_errors=self.warnings_as_errors
                                         )
                     future_to_test[executor.submit(pb.process, message)] = test.name
 

scripts/sanitycheck

@@ -412,8 +412,10 @@ Artificially long but functional example:
              "run. Intended to be run by CI when tagging an official "
              "release. This database is used as a basis for comparison "
              "when looking for deltas in metrics such as footprint")
-    parser.add_argument("-w", "--warnings-as-errors", action="store_true",
+
+    parser.add_argument("-W", "--disable-warnings-as-errors", action="store_true",
                         help="Treat warning conditions as errors")
+
     parser.add_argument(
         "-v",
         "--verbose",
@@ -799,6 +801,7 @@ def main():
     suite.west_flash = options.west_flash
     suite.west_runner = options.west_runner
     suite.verbose = VERBOSE
+    suite.warnings_as_errors = not options.disable_warnings_as_errors
 
     if options.ninja:
         suite.generator_cmd = "ninja"