fix lint

Author: sbryngelson

toolchain/mfc/build.py

@@ -37,77 +37,75 @@ def __hash__(self) -> int:
         return hash(self.name)
 
     def get_compiler_info(self) -> str:
-      with tempfile.TemporaryDirectory() as build_dir:
-          subprocess.run(
-              ["cmake", "-B", build_dir, "-S", ".", "-DCMAKE_VERBOSE_MAKEFILE=ON"],
-              check=True,
-              stdout=subprocess.DEVNULL,
-              stderr=subprocess.DEVNULL,
-          )
-
-          cache = pathlib.Path(build_dir) / "CMakeCache.txt"
-          if not cache.exists():
-              return "unknown"
-
-          pat = re.compile(r"CMAKE_.*_COMPILER:FILEPATH=(.+)", re.IGNORECASE)
-          compiler_name = ""
-
-          for line in cache.read_text(errors="ignore").splitlines():
-              if "fortran" in line.lower():
-                  m = pat.search(line)
-                  if m:
-                      compiler_name = pathlib.Path(m.group(1).strip()).name
-                      break
-
-      name = compiler_name.lower()
-      if "gfortran" in name:
-          return "gnu"
-      if "ifort" in name or "ifx" in name:
-          return "intel"
-      if "nvfortran" in name:
-          return "nvhpc"
-      if name == "ftn" or "cray" in name:
-          return "cray"
-      if "pgfortran" in name:
-          return "pgi"
-      if "clang" in name:
-          return "clang"
-      if "flang" in name:
-          return "flang"
-      return "unknown"
+        compiler = "unknown"
+        with tempfile.TemporaryDirectory() as build_dir:
+            subprocess.run(
+                ["cmake", "-B", build_dir, "-S", ".", "-DCMAKE_VERBOSE_MAKEFILE=ON"],
+                check=True,
+                stdout=subprocess.DEVNULL,
+                stderr=subprocess.DEVNULL,
+            )
+
+            cache = pathlib.Path(build_dir) / "CMakeCache.txt"
+            if not cache.exists():
+                compiler = "unknown"
+
+            pat = re.compile(r"CMAKE_.*_COMPILER:FILEPATH=(.+)", re.IGNORECASE)
+            compiler_name = ""
+
+            for line in cache.read_text(errors="ignore").splitlines():
+                if "fortran" in line.lower():
+                    m = pat.search(line)
+                    if m:
+                        compiler_name = pathlib.Path(m.group(1).strip()).name
+                        break
+
+        name = compiler_name.lower()
+        if "gfortran" in name:
+            compiler = "gnu"
+        if "ifort" in name or "ifx" in name:
+            compiler = "intel"
+        if "nvfortran" in name:
+            compiler = "nvhpc"
+        if name == "ftn" or "cray" in name:
+            compiler = "cray"
+        if "pgfortran" in name:
+            compiler = "pgi"
+        if "clang" in name:
+            compiler = "clang"
+        if "flang" in name:
+            compiler = "flang"
+        return compiler
 
     def get_slug(self, case: Case ) -> str:
         if self.isDependency:
             return self.name
 
         # Start with target name
         parts = [self.name]
-        
+
         # Add active configuration options
         cfg = CFG()
         cfg_parts = []
         for key, value in sorted(cfg.items()):
             if value:  # Only include enabled options
                 cfg_parts.append(key)
-        
+
         if cfg_parts:
             parts.append('-'.join(cfg_parts))
-        
+
         # Add chemistry info if enabled
         if case.params.get('chemistry', 'F') == 'T':
             parts.append(f"chem-{case.get_cantera_solution().name}")
-        
+
         # Add case optimization if enabled
         if case.params.get('case_optimization', False):
             parts.append('opt')
-            
+
         # Add compiler identifier
-        try:
-          compiler_id = self.get_compiler_info()
-          if compiler_id and compiler_id != 'unknown':
-              parts.append(f"fc-{compiler_id}")
-        except:
-          parts.append(f"fc-unknown")
+        compiler_id = self.get_compiler_info()
+        if compiler_id:
+            parts.append(f"fc-{compiler_id}")
 
         # Join all parts with underscores
         return '_'.join(parts)