Update pre-commit and extract a function (#43)

* Update build script

* Pre-commit changes

Author: kratman

.pre-commit-config.yaml

@@ -4,7 +4,7 @@ ci:
 
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.1.3"
+    rev: "v0.11.9"
     hooks:
       - id: ruff
         args: [--fix, --show-fixes]

install_KLU_Sundials.py

@@ -60,19 +60,19 @@ def install_suitesparse(download_dir):
                 # if in CI, set RPATH to the install directory for SuiteSparse_config
                 # dylibs to find libomp.dylib when repairing the wheel
                 if os.environ.get("CIBUILDWHEEL") == "1":
-                    env[
-                        "CMAKE_OPTIONS"
-                    ] = f"-DCMAKE_INSTALL_PREFIX={install_dir} -DCMAKE_INSTALL_RPATH={install_dir}/lib"
+                    env["CMAKE_OPTIONS"] = (
+                        f"-DCMAKE_INSTALL_PREFIX={install_dir} -DCMAKE_INSTALL_RPATH={install_dir}/lib"
+                    )
                 else:
                     env["CMAKE_OPTIONS"] = f"-DCMAKE_INSTALL_PREFIX={install_dir}"
             else:
                 # For AMD, COLAMD, BTF and KLU; do not set a BUILD RPATH but use an
                 # INSTALL RPATH in order to ensure that the dynamic libraries are found
                 # at runtime just once. Otherwise, delocate complains about multiple
                 # references to the SuiteSparse_config dynamic library (auditwheel does not).
-                env[
-                    "CMAKE_OPTIONS"
-                ] = f"-DCMAKE_INSTALL_PREFIX={install_dir} -DCMAKE_INSTALL_RPATH={install_dir}/lib -DCMAKE_INSTALL_RPATH_USE_LINK_PATH=FALSE -DCMAKE_BUILD_WITH_INSTALL_RPATH=FALSE"
+                env["CMAKE_OPTIONS"] = (
+                    f"-DCMAKE_INSTALL_PREFIX={install_dir} -DCMAKE_INSTALL_RPATH={install_dir}/lib -DCMAKE_INSTALL_RPATH_USE_LINK_PATH=FALSE -DCMAKE_BUILD_WITH_INSTALL_RPATH=FALSE"
+                )
             subprocess.run(make_cmd, cwd=build_dir, env=env, shell=True, check=True)
             subprocess.run(install_cmd, cwd=build_dir, check=True)
 
@@ -269,14 +269,7 @@ def parallel_download(urls, download_dir):
                 future.result()
 
     # First check requirements: make and cmake
-    try:
-        subprocess.run(["make", "--version"])
-    except OSError as error:
-        raise RuntimeError("Make must be installed.") from error
-    try:
-        subprocess.run(["cmake", "--version"])
-    except OSError as error:
-        raise RuntimeError("CMake must be installed.") from error
+    check_build_tools()
 
     # Build in parallel wherever possible
     os.environ["CMAKE_BUILD_PARALLEL_LEVEL"] = str(cpu_count())
@@ -341,5 +334,16 @@ def parallel_download(urls, download_dir):
             install_suitesparse(download_dir)
 
 
+def check_build_tools():
+    try:
+        subprocess.run(["make", "--version"])
+    except OSError as error:
+        raise RuntimeError("Make must be installed.") from error
+    try:
+        subprocess.run(["cmake", "--version"])
+    except OSError as error:
+        raise RuntimeError("CMake must be installed.") from error
+
+
 if __name__ == "__main__":
     build_solvers()