Add exception when maven subprocess raises error (#182)

Signed-off-by: Jiyeong Seok <[email protected]>
fosslight · Dec 21, 2023 · 6065878 · 6065878
1 parent 7faab9b
commit 6065878
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/src/fosslight_dependency/package_manager/Maven.py b/src/fosslight_dependency/package_manager/Maven.py
@@ -146,12 +146,16 @@ def run_maven_plugin(self):
             logger.error(f"Failed to run maven plugin: {cmd}")
 
         cmd = f"{cmd_mvn} dependency:tree"
-        ret_txt = subprocess.check_output(cmd, text=True, shell=True)
-        if ret_txt is not None:
-            self.parse_dependency_tree(ret_txt)
-            self.set_direct_dependencies(True)
-        else:
-            logger.error(f"Failed to run: {cmd}")
+        try:
+            ret_txt = subprocess.check_output(cmd, text=True, shell=True)
+            if ret_txt is not None:
+                self.parse_dependency_tree(ret_txt)
+                self.set_direct_dependencies(True)
+            else:
+                logger.error(f"Failed to run: {cmd}")
+                self.set_direct_dependencies(False)
+        except Exception as e:
+            logger.error(f"Failed to run '{cmd}': {e}")
             self.set_direct_dependencies(False)
 
     def create_dep_stack(self, dep_line):