Fix style for finding external grep

JCSDA · Dec 16, 2024 · 68ca1c4 · 68ca1c4
1 parent 4665c25
commit 68ca1c4
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/var/spack/repos/builtin/packages/grep/package.py b/var/spack/repos/builtin/packages/grep/package.py
@@ -34,13 +34,17 @@ class Grep(AutotoolsPackage):
 
     @classmethod
     def determine_version(cls, exe):
-        version_string = Executable(exe)("--version", output=str, error=str).split('\n')[0]
+        version_string = Executable(exe)("--version", output=str, error=str).split("\n")[0]
+        # Linux
         if "GNU grep" in version_string:
             return version_string.lstrip("grep (GNU grep)").strip()
+        # macOS
         elif "BSD grep, GNU compatible" in version_string:
-            return version_string.lstrip("grep (BSD grep, GNU compatible)").rstrip("-FreeBSD").strip()
+            return (
+                version_string.lstrip("grep (BSD grep, GNU compatible)").rstrip("-FreeBSD").strip()
+            )
+        # Don't know how to handle this version of grep, don't add it
         else:
-            # Don't know how to handle this version of grep, don't add it
             return None
 
     def configure_args(self):