[Buildroot] [PATCH 1/1] rustc: disable for ARMV7A with EABI

[Peter Korsgaard]

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni at bootlin.com> writes:

 > Hello,
 > On Mon, 26 Feb 2018 21:04:21 +0100, Eric Le Bihan wrote:
 >> The Rust project does not support armv7a EABI, only EABIHF [1]. So
 >> disable rustc for this ARM variant.

 > Are you sure ?

 >> [1] https://forge.rust-lang.org/platform-support.html

 > On this page, I see:

 > arm-unknown-linux-gnueabi 	✓ 	✓ 	✓ 	ARMv6 Linux
 > arm-unknown-linux-gnueabihf 	✓ 	✓ 	✓ 	ARMv6 Linux, hardfloat
 > armv7-unknown-linux-gnueabihf 	✓ 	✓ 	✓ 	ARMv7 Linux

 > The arm-unknown-linux-gnueabi variant should apply to ARMv7 as well.

Ok, but that is more of an improvement. It was still wrong that it tried
to download an ARMv7 EABI toolchain.

 > It looks like a better fix would be:

 > config BR2_PACKAGE_HOST_RUSTC_ARCH_SUPPORTS
 >         bool
 >         # The pre-built Rust standard library is only available for the
 >         # following architectures/ABIs, and is built against glibc.
 >         default y if BR2_i386
 >         default y if BR2_x86_64
 >         default y if BR2_aarch64
 > -       default y if BR2_arm && !BR2_ARM_CPU_ARMV4 && !BR2_ARM_CPU_ARMV5
 > +	default y if BR2_ARM_CPU_ARMV6 || BR2_ARM_CPU_ARMV7A

I agree that having positive logic would be nicer. It is a bit more
complicated than this as we also should handle ARMV8A cores in ARM mode
similar to how ve handle ARMV7A.

Care to send an updated patch on top of master doing that?

-- 
Bye, Peter Korsgaard