Skip to contentSkip to conversation

python-numpy: disable for ARCompact

PATCH Superseded Evgeniy Didin 3 participants 3 messages Nov 8, 2017 – Nov 9, 2017

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 python-numpy: disable for ARCompact

This package fails when building for ARCompact due to toolchain issue.

Marking this with special comment "#ARC toolchain issues for ARCompact"
as the package is to be enabled as soon as the issue with the ARC toolchain is resolved. Fixes:

	http://autobuild.buildroot.net/results/4c4/4c4eaa90335756448007ae3c5ae3839fb17d4442//

The same issue appears building "trousers" package for ARCompact.

CC Alexey Brodkin <[email protected]>
CC arc-buildroot at synopsys.com
CC Thomas Petazzoni <[email protected]>
---
 package/python-numpy/Config.in | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)
package/python-numpy/Config.in +5 -3
index 859542f584..403b466417 100644
@@ -1,13 +1,15 @@
 config BR2_PACKAGE_PYTHON_NUMPY_ARCH_SUPPORTS
 	bool
 	# Numpy has some CPU specific code
-	default y if BR2_arc || BR2_aarch64 || BR2_arm || BR2_armeb || BR2_i386 \
-		|| BR2_mips || BR2_mipsel || BR2_powerpc || BR2_powerpc64 \
-		|| BR2_sh || BR2_x86_64
+	default y if (BR2_arc && !(BR2_arc770d && BR2_arc750d)) || BR2_aarch64 \
+		|| BR2_arm || BR2_armeb || BR2_i386 || BR2_mips || BR2_mipsel \
+		|| BR2_powerpc || BR2_powerpc64 || BR2_sh || BR2_x86_64
 
 config BR2_PACKAGE_PYTHON_NUMPY
 	bool "python-numpy"
 	depends on BR2_PACKAGE_PYTHON_NUMPY_ARCH_SUPPORTS
+	#ARC toolchain issues for ARCompact
+	depends on !BR2_arc770d && !BR2_arc750d
 	help
 	  NumPy is the fundamental package for scientific computing
 	  with Python.
Signature · 1 line
2.11.0
#1 lore raw

PATCH 1/1 python-numpy: disable for ARCompact

Hello,

Quoted 1 line

The same issue appears building "trousers" package for ARCompact.

But trousers is not using python-numpy. Are you going to send a similar patch for trousers ?

Quoted 5 lines · Evgeniy Didin
package/python-numpy/Config.in +0 -0
index 859542f584..403b466417 100644

package/python-numpy/Config.in line 6

Quoted 9 lines
package/python-numpy/Config.in +3 -3
@@ -1,13 +1,15 @@
 config BR2_PACKAGE_PYTHON_NUMPY_ARCH_SUPPORTS
 	bool
 	# Numpy has some CPU specific code
-	default y if BR2_arc || BR2_aarch64 || BR2_arm || BR2_armeb || BR2_i386 \
-		|| BR2_mips || BR2_mipsel || BR2_powerpc || BR2_powerpc64 \
-		|| BR2_sh || BR2_x86_64
+	default y if (BR2_arc && !(BR2_arc770d && BR2_arc750d)) || BR2_aarch64 \
+		|| BR2_arm || BR2_armeb || BR2_i386 || BR2_mips || BR2_mipsel \
+		|| BR2_powerpc || BR2_powerpc64 || BR2_sh || BR2_x86_64

This is getting ugly. It would be nice to have a first patch (separate from your addition) that reformats to:

default y if BR2_arc
default y if BR2_aarch64
default y if BR2_arm
default y if BR2_armeb
...

And then your patch would be:

- default y if BR2_arc
+ default y if BR2_arc && !BR2_arc770d && !BR2_arc750d

package/python-numpy/Config.in line 12

Quoted 6 lines
package/python-numpy/Config.in +2 -0
 config BR2_PACKAGE_PYTHON_NUMPY
 	bool "python-numpy"
 	depends on BR2_PACKAGE_PYTHON_NUMPY_ARCH_SUPPORTS
+	#ARC toolchain issues for ARCompact
+	depends on !BR2_arc770d && !BR2_arc750d

Why are you duplicating this here? You've already excluded the problematic ARC architectures from BR2_PACKAGE_PYTHON_NUMPY_ARCH_SUPPORTS (which is the right thing to do), and BR2_PACKAGE_PYTHON_NUMPY depends on BR2_PACKAGE_PYTHON_NUMPY_ARCH_SUPPORTS.

Could you rework your patch accordingly?

Thanks a lot!

Thomas

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

PATCH 1/1 python-numpy: disable for ARCompact

Hi Evgeniy,

Quoted 7 lines · Thomas Petazzoni

Hello,

Quoted 2 lines

The same issue appears building "trousers" package for ARCompact.

But trousers is not using python-numpy. Are you going to send a similar patch for trousers ?

I just realized that indeed "trousers" package disabled for any ARC, which is obviously incorrect, given it's only ARCompact tools are affected.

Could you please send similar patch for trousers so in case of ARCv2 it becomes available for ARC.

-Alexey