openblas: use cmake

Author: ywwry66

Formula/o/openblas.rb

@@ -1,8 +1,6 @@
 class Openblas < Formula
   desc "Optimized BLAS library"
   homepage "https://www.openblas.net/"
-  url "https://github.com/OpenMathLib/OpenBLAS/archive/refs/tags/v0.3.29.tar.gz"
-  sha256 "38240eee1b29e2bde47ebb5d61160207dc68668a54cac62c076bb5032013b1eb"
   # The main license is BSD-3-Clause. Additionally,
   # 1. OpenBLAS is based on GotoBLAS2 so some code is under original BSD-2-Clause-Views
   # 2. lapack-netlib/ is a bundled LAPACK so it is BSD-3-Clause-Open-MPI
@@ -11,6 +9,29 @@ class Openblas < Formula
   license all_of: ["BSD-3-Clause", "BSD-2-Clause-Views", "BSD-3-Clause-Open-MPI", "BSD-2-Clause"]
   head "https://github.com/OpenMathLib/OpenBLAS.git", branch: "develop"
 
+  stable do
+    url "https://github.com/OpenMathLib/OpenBLAS/archive/refs/tags/v0.3.29.tar.gz"
+    sha256 "38240eee1b29e2bde47ebb5d61160207dc68668a54cac62c076bb5032013b1eb"
+
+    # Remove patch in the next release
+    patch do
+      url "https://github.com/OpenMathLib/OpenBLAS/commit/70865a894e5963724ff9ccb05323ac2234a918b9.patch?full_index=1"
+      sha256 "80beeb28e5baf21af05ac114a1cca97855faf298e40b3b290005977fad260f52"
+    end
+
+    # Remove patch in the next release
+    patch do
+      url "https://github.com/OpenMathLib/OpenBLAS/commit/d659f3c3f6058ece8a704423b72ece3821cfba4d.patch?full_index=1"
+      sha256 "88ad7b0802e914535dcb8ceedfaec6cc01d379d2029111c29a7b55176a511f88"
+    end
+
+    # Remove patch in the next release
+    patch do
+      url "https://github.com/OpenMathLib/OpenBLAS/commit/9aa7a0b2a7b2770adec6ff26b34660d3bcd8c49c.patch?full_index=1"
+      sha256 "0538ce7223ec5813c80e148388e892184b5e5e396c0868f903c8c0c5e19c5c2e"
+    end
+  end
+
   livecheck do
     url :stable
     strategy :github_latest
@@ -28,21 +49,18 @@ class Openblas < Formula
 
   keg_only :shadowed_by_macos, "macOS provides BLAS in Accelerate.framework"
 
+  depends_on "cmake" => :build
+  depends_on "pkgconf" => :test
   depends_on "gcc" # for gfortran
-  fails_with :clang
+
+  on_macos do
+    depends_on "libomp"
+  end
 
   def install
     ENV.runtime_cpu_detection
-    ENV.deparallelize # build is parallel by default, but setting -j confuses it
-
-    # The build log has many warnings of macOS build version mismatches.
-    ENV["MACOSX_DEPLOYMENT_TARGET"] = MacOS.version.to_s if OS.mac?
-    ENV["DYNAMIC_ARCH"] = "1"
-    ENV["USE_OPENMP"] = "1"
-    # Force a large NUM_THREADS to support larger Macs than the VMs that build the bottles
-    ENV["NUM_THREADS"] = "56"
-    # See available targets in TargetList.txt
-    ENV["TARGET"] = case Hardware.oldest_cpu
+
+    target = case Hardware.oldest_cpu
     when :arm_vortex_tempest
       "VORTEX"
     when :westmere
@@ -51,16 +69,30 @@ def install
       Hardware.oldest_cpu.upcase.to_s
     end
 
-    # Apple Silicon does not support SVE
-    # https://github.com/xianyi/OpenBLAS/issues/4212
-    ENV["NO_SVE"] = "1" if Hardware::CPU.arm?
+    args = %W[
+      -DUSE_OPENMP=ON
+      -DBUILD_SHARED_LIBS=ON
+      -DBUILD_STATIC_LIBS=ON
+      -DNUM_THREADS=64
+      -DTARGET=#{target}
+    ]
+
+    args << "-DDYNAMIC_ARCH=ON" if !OS.mac? || Hardware::CPU.intel?
 
-    # Must call in two steps
-    system "make", "CC=#{ENV.cc}", "FC=gfortran", "libs", "netlib", "shared"
-    system "make", "PREFIX=#{prefix}", "install"
+    if OS.mac?
+      libomp = Formula["libomp"]
+      args << "-DOpenMP_Fortran_LIB_NAMES=omp"
+      args << "-DOpenMP_omp_LIBRARY=#{libomp.opt_lib}/libomp.dylib"
+    end
+
+    system "cmake", "-S", ".", "-B", "build", *args, *std_cmake_args
+    system "cmake", "--build", "build"
+    system "cmake", "--install", "build"
 
     lib.install_symlink shared_library("libopenblas") => shared_library("libblas")
     lib.install_symlink shared_library("libopenblas") => shared_library("liblapack")
+
+    inreplace lib/"pkgconfig/openblas.pc", prefix, opt_prefix
   end
 
   test do
@@ -85,8 +117,10 @@ def install
         return 0;
       }
     C
-    system ENV.cc, "test.c", "-I#{include}", "-L#{lib}", "-lopenblas",
-                   "-o", "test"
+
+    ENV.prepend_path "PKG_CONFIG_PATH", lib/"pkgconfig"
+    flags = shell_output("pkgconf --cflags --libs openblas").chomp.split
+    system ENV.cc, "test.c", "-o", "test", *flags
     system "./test"
   end
 end