BLD: fix _umath_linalg dependencies (#24584)

steppi · rgommers · web-flow · commit a115ed3a3c92 · 2023-08-30T14:20:43.000+02:00
Closes gh-24512, where `linalg.eigvalsh` was observed to be non-thread safe. This was due to the non-thread safe `lapack_lite` being called instead of the installed BLAS/LAPACK. Co-authored-by: Ralf Gommers <ralf.gommers@gmail.com>
diff --git a/numpy/linalg/meson.build b/numpy/linalg/meson.build
@@ -1,34 +1,37 @@
+# Note that `python_xerbla.c` was excluded on Windows in setup.py;
+# unclear why and it seems needed, so unconditionally used here.
 lapack_lite_sources = [
-  'lapack_lite/f2c.c',
-  'lapack_lite/f2c_c_lapack.c',
-  'lapack_lite/f2c_d_lapack.c',
-  'lapack_lite/f2c_s_lapack.c',
-  'lapack_lite/f2c_z_lapack.c',
-  'lapack_lite/f2c_blas.c',
-  'lapack_lite/f2c_config.c',
-  'lapack_lite/f2c_lapack.c',
   'lapack_lite/python_xerbla.c',
 ]
-
-# TODO: ILP64 support
-
-lapack_lite_module_src = ['lapack_litemodule.c']
 if not have_lapack
-  lapack_lite_module_src += lapack_lite_sources
+  lapack_lite_sources += [
+    'lapack_lite/f2c.c',
+    'lapack_lite/f2c_c_lapack.c',
+    'lapack_lite/f2c_d_lapack.c',
+    'lapack_lite/f2c_s_lapack.c',
+    'lapack_lite/f2c_z_lapack.c',
+    'lapack_lite/f2c_blas.c',
+    'lapack_lite/f2c_config.c',
+    'lapack_lite/f2c_lapack.c',
+  ]
 endif
 
 py.extension_module('lapack_lite',
-  lapack_lite_module_src,
+  [
+    'lapack_litemodule.c',
+    lapack_lite_sources,
+  ],
   dependencies: [np_core_dep, blas_dep, lapack_dep],
   install: true,
   subdir: 'numpy/linalg',
 )
 
-_umath_linalg_src = ['umath_linalg.cpp'] + lapack_lite_sources
-
 py.extension_module('_umath_linalg',
-  _umath_linalg_src,
-  dependencies: np_core_dep,
+  [
+    'umath_linalg.cpp',
+    lapack_lite_sources,
+  ],
+  dependencies: [np_core_dep, blas_dep, lapack_dep],
   link_with: npymath_lib,
   install: true,
   subdir: 'numpy/linalg',
