scripts: west_command: spdx: Exit with non-zero code on failure

If the init or the generation fails, the resulting exit code should be
non-zero.

Signed-off-by: Pieter De Gendt <[email protected]>

Author: pdgendt

scripts/west_commands/spdx.py

@@ -74,8 +74,9 @@ def do_run_init(self, args):
         if query_ready:
             self.inf("initialized; run `west build` then run `west spdx`")
         else:
-            self.err("Couldn't create CMake file-based API query directory")
-            self.err("You can manually create an empty file at $BUILDDIR/.cmake/api/v1/query/codemodel-v2")
+            self.die("Couldn't create CMake file-based API query directory\n"
+                     "You can manually create an empty file at "
+                     "$BUILDDIR/.cmake/api/v1/query/codemodel-v2")
 
     def do_run_spdx(self, args):
         if not args.build_dir:
@@ -110,4 +111,5 @@ def do_run_spdx(self, args):
             # create the directory
             os.makedirs(cfg.spdxDir, exist_ok=False)
 
-        makeSPDX(cfg)
+        if not makeSPDX(cfg):
+            self.die("Failed to create SPDX output")