Add checking for unused compilers to software-definitions

This commit adds a check to the software-definitions command to ensure
compilers are used within the applications they are defined in.

Author: douglasjacobsen

lib/ramble/ramble/cmd/software_definitions.py

@@ -18,6 +18,7 @@
 section = "developer"
 level = "long"
 
+unused_compilers = {}
 definitions = {}
 conflicts = {}
 used_by = {}
@@ -56,7 +57,12 @@ def collect_definitions():
     """
     top_level_attrs = ["compilers", "software_specs"]
 
-    types_to_print = [ramble.repository.ObjectTypes.applications]
+    types_to_print = [
+        ramble.repository.ObjectTypes.applications,
+        ramble.repository.ObjectTypes.modifiers,
+        ramble.repository.ObjectTypes.workflow_managers,
+        ramble.repository.ObjectTypes.package_managers,
+    ]
 
     for object_type in types_to_print:
         obj_path = ramble.repository.paths[object_type]
@@ -65,6 +71,20 @@ def collect_definitions():
 
             obj_namespace = f"{obj_repo.full_namespace}.{obj_inst.name}"
 
+            for compiler_name in obj_inst.compilers:
+                used = False
+                for pkg_defs in obj_inst.software_specs.values():
+                    for pkg_def in pkg_defs:
+                        if (
+                            getattr(pkg_def, "compiler", "__rmb_no_compiler_attr__")
+                            == compiler_name
+                        ):
+                            used = True
+                if not used:
+                    if compiler_name not in unused_compilers:
+                        unused_compilers[compiler_name] = []
+                    unused_compilers[compiler_name].append(obj_namespace)
+
             for top_level_attr in top_level_attrs:
                 if hasattr(obj_inst, top_level_attr):
                     for pkg_name, pkg_defs in getattr(obj_inst, top_level_attr).items():
@@ -110,26 +130,36 @@ def count_conflicts():
     num_conflicts = 0
     for pkg_name in conflicts:
         num_conflicts += len(conflicts[pkg_name])
+    for object_names in unused_compilers.values():
+        num_conflicts += len(object_names)
     return num_conflicts
 
 
 def print_conflicts():
     """Print conflict information, if any exist"""
-    if len(conflicts) > 0:
-        color.cprint(section_title("Software Definition Conflicts:"))
-        for pkg_name in conflicts:
-
-            color.cprint(f'{nested_1("Package")}: {pkg_name}:')
-            color.cprint("\tDefined as:")
-            for attr in ["pkg_spec", "compiler_spec", "compiler"]:
-                if hasattr(definitions[pkg_name], attr):
-                    attr_def = getattr(definitions[pkg_name], attr)
-                    if attr_def:
-                        color.cprint(f'\t\t{attr} = {attr_def.replace("@", "@@")}')
-            color.cprint("\tIn objects:")
-            colify(used_by[pkg_name], indent=24, output=sys.stdout)
-            color.cprint("\tConflicts with objects:")
-            colify(conflicts[pkg_name], indent=24, output=sys.stdout)
+    if len(conflicts) > 0 or len(unused_compilers) > 0:
+        if len(conflicts) > 0:
+            color.cprint(section_title("Software Definition Conflicts:"))
+            for pkg_name in conflicts:
+
+                color.cprint(f'{nested_1("Package")}: {pkg_name}:')
+                color.cprint("\tDefined as:")
+                for attr in ["pkg_spec", "compiler_spec", "compiler"]:
+                    if hasattr(definitions[pkg_name], attr):
+                        attr_def = getattr(definitions[pkg_name], attr)
+                        if attr_def:
+                            color.cprint(f'\t\t{attr} = {attr_def.replace("@", "@@")}')
+                color.cprint("\tIn objects:")
+                colify(used_by[pkg_name], indent=24, output=sys.stdout)
+                color.cprint("\tConflicts with objects:")
+                colify(conflicts[pkg_name], indent=24, output=sys.stdout)
+
+        if len(unused_compilers) > 0:
+            color.cprint(section_title("Unused Compilers:"))
+            for compiler_name, object_names in unused_compilers.items():
+                color.cprint(nested_1(f"    Compiler {compiler_name} is not used in packages:"))
+                colify(object_names, indent=8, output=sys.stdout)
+                color.cprint("\n")
     else:
         color.cprint(section_title("No Conflicts Detected"))
 
@@ -164,5 +194,8 @@ def software_definitions(parser, args, unknown_args):
 
     if args.error_on_conflict:
         num_conflicts = count_conflicts()
-        color.cprint(f"{num_conflicts} conflicts detected.")
+        if num_conflicts == 1:
+            color.cprint(f"{num_conflicts} conflict detected.")
+        else:
+            color.cprint(f"{num_conflicts} conflicts detected.")
         sys.exit(num_conflicts)