Skip to contentSkip to conversation

package/ola: fix missing numpy

PATCH New Jörg Krause 2 participants 2 messages Nov 24, 2016 – Nov 25, 2016

lore
Share & tools
lore

Jörg Krause

author
#0 lore raw

PATCH 1/1 package/ola: fix missing numpy

OLAs RDM test option needs NumPy.

Note that we need to pass ac_cv_have_pymod_numpy=yes as NumPy builds some shared libraries and ola checks for NumPy using a host Python test program which fails with 'wrong ELF class'.

Fixes:
http://autobuild.buildroot.net/results/750/750c9bcc753e2c1b7b05d6c43fabf4c405e15f04/
http://autobuild.buildroot.net/results/24a/24ada1e0ec695cb12028048728a9d1e2bcf75a03/
http://autobuild.buildroot.net/results/111/111c5f3eed6a5931de27a1b0b02be6e2114198e6/
.. and more.

Signed-off-by J?rg Krause <[email protected]>
---
 package/ola/Config.in | 2 ++
 package/ola/ola.mk    | 4 ++++
 2 files changed, 6 insertions(+)
package/ola/Config.in +2 -0
index 7b56f5c..2f1d33e 100644
@@ -54,6 +54,8 @@ config BR2_PACKAGE_OLA_EXAMPLES
 config BR2_PACKAGE_OLA_RDM_TESTS
 	bool "rdm tests"
 	depends on BR2_PACKAGE_OLA_PYTHON_BINDINGS
+	depends on BR2_PACKAGE_PYTHON_NUMPY_ARCH_SUPPORTS
+	select BR2_PACKAGE_PYTHON_NUMPY
 	help
 	  Build OLA RDM tests.
package/ola/ola.mk +4 -0
index 75ab25e..24e53e4 100644
@@ -87,6 +87,10 @@ endif
 
 ifeq ($(BR2_PACKAGE_OLA_RDM_TESTS),y)
 OLA_CONF_OPTS += --enable-rdm-tests
+OLA_DEPENDENCIES += python-numpy
+# needed as numpy builds some shared libraries and ola checks for numpy
+# using a host python test program which fails with 'wrong ELF class'.
+OLA_CONF_ENV = ac_cv_have_pymod_numpy=yes
 else
 OLA_CONF_OPTS += --disable-rdm-tests
 endif
Signature · 1 line
2.10.2
Applied
#1 lore raw

PATCH 1/1 package/ola: fix missing numpy

Hello,

Quoted 12 lines · J?rg Krause

OLAs RDM test option needs NumPy.

Note that we need to pass ac_cv_have_pymod_numpy=yes as NumPy builds some shared libraries and ola checks for NumPy using a host Python test program which fails with 'wrong ELF class'.

Fixes:
http://autobuild.buildroot.net/results/750/750c9bcc753e2c1b7b05d6c43fabf4c405e15f04/
http://autobuild.buildroot.net/results/24a/24ada1e0ec695cb12028048728a9d1e2bcf75a03/
http://autobuild.buildroot.net/results/111/111c5f3eed6a5931de27a1b0b02be6e2114198e6/
.. and more.

Quoted 5 lines
Signed-off-by J?rg Krause <[email protected]>
---
 package/ola/Config.in | 2 ++
 package/ola/ola.mk    | 4 ++++
 2 files changed, 6 insertions(+)

Applied to master, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com