diff mbox series

[v5,2/2] package/python-scipy: new package

Message ID 20190930084326.4414-2-esben@geanix.com
State Superseded
Headers show
Series [v5,1/2] package/python-numpy: fixup numpy distutils for cross compilation | expand

Commit Message

Esben Haabendal Sept. 30, 2019, 8:43 a.m. UTC 
See http://scipy.github.io/devdocs/building/linux.html
for SciPy build requirements:

    * BLAS and LAPACK libraries
    * C and Fortran compilers
    * Python header files
    * Cython compiler

For now, clapack package is used to provide both BLAS and LAPACK libraries, as
that is what is supported by the current python-numpy recipe which
python-scipy depends on.

With additional work, it should be possible to improve both python-numpy and
python-scipy to support other BLAS and LAPACK library implementations, but it
is probably best to keep such changes aligned.

A site.cfg file is used to configure numpy distutils extension to find include
files and libraries in stagging.

Note, host-python-numpy is needed in order to use numpy distutils extension
from it.

Signed-off-by: Esben Haabendal <esben@geanix.com>
---
 DEVELOPERS                             |  1 +
 package/Config.in                      |  1 +
 package/python-scipy/Config.in         | 23 +++++++++++++++
 package/python-scipy/python-scipy.hash |  9 ++++++
 package/python-scipy/python-scipy.mk   | 40 ++++++++++++++++++++++++++
 5 files changed, 74 insertions(+)
 create mode 100644 package/python-scipy/Config.in
 create mode 100644 package/python-scipy/python-scipy.hash
 create mode 100644 package/python-scipy/python-scipy.mk

Comments

Jagan Teki Jan. 7, 2020, 5:16 p.m. UTC | #1 
On Mon, Sep 30, 2019 at 2:13 PM Esben Haabendal <esben@geanix.com> wrote:
>
> See http://scipy.github.io/devdocs/building/linux.html
> for SciPy build requirements:
>
>     * BLAS and LAPACK libraries
>     * C and Fortran compilers
>     * Python header files
>     * Cython compiler
>
> For now, clapack package is used to provide both BLAS and LAPACK libraries, as
> that is what is supported by the current python-numpy recipe which
> python-scipy depends on.
>
> With additional work, it should be possible to improve both python-numpy and
> python-scipy to support other BLAS and LAPACK library implementations, but it
> is probably best to keep such changes aligned.
>
> A site.cfg file is used to configure numpy distutils extension to find include
> files and libraries in stagging.
>
> Note, host-python-numpy is needed in order to use numpy distutils extension
> from it.
>
> Signed-off-by: Esben Haabendal <esben@geanix.com>
> ---

Thanks for the great work.

Tested the build via arm64 configuration and verified import in sample
python test.

Tested-by: Jagan Teki <jagan@amarulasolutions.com>
diff mbox series

Patch

diff --git a/DEVELOPERS b/DEVELOPERS
index 8cbf55fd1013..499fa5954ffe 100644
--- a/DEVELOPERS
+++ b/DEVELOPERS
@@ -727,6 +727,7 @@  F:	package/szip/
 N:	Esben Haabendal <esben@haabendal.dk>
 F:	boot/gummiboot/
 F:	package/python-kiwisolver/
+F:	package/python-scipy/
 
 N:	Etienne Carriere <etienne.carriere@linaro.org>
 F:	boot/optee-os/
diff --git a/package/Config.in b/package/Config.in
index 713ef79edb9e..1a98ad9b2bcb 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -1059,6 +1059,7 @@  menu "External python modules"
 	source "package/python-scapy/Config.in"
 	source "package/python-scapy3k/Config.in"
 	source "package/python-schedule/Config.in"
+	source "package/python-scipy/Config.in"
 	source "package/python-sdnotify/Config.in"
 	source "package/python-secretstorage/Config.in"
 	source "package/python-see/Config.in"
diff --git a/package/python-scipy/Config.in b/package/python-scipy/Config.in
new file mode 100644
index 000000000000..071ceb2b70a6
--- /dev/null
+++ b/package/python-scipy/Config.in
@@ -0,0 +1,23 @@ 
+config BR2_PACKAGE_PYTHON_SCIPY
+	bool "python-scipy"
+	depends on BR2_PACKAGE_PYTHON_NUMPY_ARCH_SUPPORTS
+	depends on !BR2_powerpc || BR2_TOOLCHAIN_USES_GLIBC # clapack
+        depends on !BR2_m68k_cf # clapack
+	depends on BR2_TOOLCHAIN_USES_GLIBC || BR2_TOOLCHAIN_USES_MUSL # python-numpy
+	depends on BR2_TOOLCHAIN_HAS_FORTRAN
+	depends on BR2_PACKAGE_PYTHON3
+	select BR2_PACKAGE_HOST_PYTHON_CYTHON
+	select BR2_PACKAGE_PYTHON_NUMPY
+	select BR2_PACKAGE_CLAPACK
+	help
+	  The SciPy library is one of the core packages that make up the SciPy
+	  stack. It provides many user-friendly and efficient numerical
+	  routines such as routines for numerical integration, interpolation,
+	  optimization, linear algebra and statistics.
+
+	  https://www.scipy.org/scipylib/
+
+comment "python-scipy needs toolchain w/ fortran and glibc or musl"
+	depends on BR2_PACKAGE_PYTHON3
+	depends on !BR2_TOOLCHAIN_HAS_FORTRAN || \
+		(!BR2_TOOLCHAIN_USES_GLIBC && !BR2_TOOLCHAIN_USES_MUSL)
diff --git a/package/python-scipy/python-scipy.hash b/package/python-scipy/python-scipy.hash
new file mode 100644
index 000000000000..6b0c3605a3b8
--- /dev/null
+++ b/package/python-scipy/python-scipy.hash
@@ -0,0 +1,9 @@ 
+# Locally generated
+sha256 4ea68de2840cf7d35c58464412d21f6f154cab7fba610c11002603ee0b9e2372  python-scipy-1.3.1.tar.gz
+sha256 904942f7df1dc0f4a48af8d72405854fd96497bd1fe9ddb2c69d1797c22dfcd7  LICENSE.txt
+sha256 37e64a498894ac7c3b070023e3689e954a8ecf8a23b90968d09a455f1b4f7b35  scipy/linalg/src/lapack_deprecations/LICENSE
+sha256 606209a000716c5f66e33e180ce08434b96ed17db4975ab9723c6b5fbcc89609  scipy/ndimage/LICENSE.txt
+sha256 3df9207af2fdb861af0ae3b22026f163b9bcfab4e525dc4943afe2ffa3f77624  scipy/optimize/tnc/LICENSE
+sha256 f0cedf52503b2d42b83411a0a16e6fefac346dfad8fddc66f50050150123470c  scipy/sparse/linalg/dsolve/SuperLU/License.txt
+sha256 0926566f9f406178d1214f8cc796e166b1213dd7c05e0c5b461a8b8ac9e50bbe  scipy/sparse/linalg/eigen/arpack/ARPACK/COPYING
+sha256 51a5a08d537b34febb851c761496ead461f90b6c3c46a5248780d63870124e9c  scipy/spatial/qhull_src/COPYING.txt
diff --git a/package/python-scipy/python-scipy.mk b/package/python-scipy/python-scipy.mk
new file mode 100644
index 000000000000..bad392121da6
--- /dev/null
+++ b/package/python-scipy/python-scipy.mk
@@ -0,0 +1,40 @@ 
+################################################################################
+#
+# python-scipy
+#
+################################################################################
+
+PYTHON_SCIPY_VERSION = 1.3.1
+PYTHON_SCIPY_SITE = $(call github,scipy,scipy,v$(PYTHON_SCIPY_VERSION))
+PYTHON_SCIPY_LICENSE = BSD-3-Clause, BSD-2-Clause, BSD, BSD-Style, \
+	Apache-2.0, MIT
+PYTHON_SCIPY_LICENSE_FILES = \
+	LICENSE.txt \
+	scipy/linalg/src/lapack_deprecations/LICENSE \
+	scipy/ndimage/LICENSE.txt \
+	scipy/optimize/tnc/LICENSE \
+	scipy/sparse/linalg/dsolve/SuperLU/License.txt \
+	scipy/sparse/linalg/eigen/arpack/ARPACK/COPYING \
+	scipy/spatial/qhull_src/COPYING.txt
+PYTHON_SCIPY_SETUP_TYPE = setuptools
+PYTHON_SCIPY_DEPENDENCIES += host-python-cython host-python-numpy python-numpy \
+	clapack
+
+PYTHON_SCIPY_BUILD_OPTS = config_fc --fcompiler=gnu95
+PYTHON_SCIPY_ENV += F90=$(TARGET_CROSS)gfortran F77=$(TARGET_CROSS)gfortran
+
+# Provide system configuration options to numpy distutils extenions, telling
+# to find all include files and libraries in staging directory.
+define PYTHON_SCIPY_CONFIGURE_CMDS
+	-rm -f $(@D)/site.cfg
+	echo "[DEFAULT]" >> $(@D)/site.cfg
+	echo "library_dirs = $(STAGING_DIR)/usr/lib" >> $(@D)/site.cfg
+	echo "include_dirs = $(STAGING_DIR)/usr/include" >> $(@D)/site.cfg
+endef
+
+# Use the target numpy pkg-config configuration files modified for target
+# cross-compilation.  Without this, numpy distutils will cause the linker to
+# link with host libnpymath.a.
+PYTHON_SCIPY_ENV += NPY_PKG_CONFIG_PATH=$(PYTHON_NUMPY_NPY_PKG_CONFIG_PATH)
+
+$(eval $(python-package))