From: Yann E. MORIN Date: Mon, 19 Oct 2015 21:59:07 +0000 (+0200) Subject: package/nodejs: fix architectural dependencies on ARM X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=f443386c5de43791b87339954783acdb7023442e;p=buildroot.git package/nodejs: fix architectural dependencies on ARM On ARM, starting with v0.12.x, the V8 JS engine is now requires at least an armv6 and at least a VFPv2. Since we're about to introduce the v4.x version, which has the same requirements, introduce an intermediate variable to hold that condition. Signed-off-by: "Yann E. MORIN" Cc: Jörg Krause Cc: Martin Bark Cc: Thomas Petazzoni Signed-off-by: Thomas Petazzoni --- diff --git a/package/nodejs/Config.in b/package/nodejs/Config.in index 329d270762..034557b480 100644 --- a/package/nodejs/Config.in +++ b/package/nodejs/Config.in @@ -25,6 +25,15 @@ comment "nodejs needs a toolchain w/ C++, dynamic library, threads" if BR2_PACKAGE_NODEJS +# Starting with 0.12.x, on ARM, V8 (the JS engine) +# now requires an armv6+ and a VFPv2+. +config BR2_PACKAGE_NODEJS_V8_ARCH_SUPPORTS + bool + # On supported architectures other than ARM, no special requirement + default y if !BR2_arm + # On ARM, at least ARMv6+ with VFPv2+ is needed + default y if !BR2_ARM_CPU_ARMV5 && BR2_ARM_CPU_HAS_VFPV2 + choice prompt "Node.js version" default BR2_BR2_PACKAGE_NODEJS_0_10_X if BR2_ARM_CPU_ARMV5 @@ -35,10 +44,9 @@ choice config BR2_BR2_PACKAGE_NODEJS_0_10_X bool "v0.10.40" -# V8 included with v0.12.5 requires at least ARMv6 config BR2_BR2_PACKAGE_NODEJS_0_12_X bool "v0.12.7" - depends on !BR2_ARM_CPU_ARMV5 + depends on BR2_PACKAGE_NODEJS_V8_ARCH_SUPPORTS endchoice