[Buildroot] [PATCH] erlang: enable host-erlang for x86/x86_64 architectures only

[Buildroot] [PATCH] erlang: enable host-erlang for x86/x86_64 architectures only

[Johan Oudinet]

And make erlang depends on host-erlang.

Signed-off-by: Johan Oudinet <johan.oudinet@gmail.com>
---
 package/erlang/Config.in | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/package/erlang/Config.in b/package/erlang/Config.in
index 96af5516fd..bf745555e8 100644
--- a/package/erlang/Config.in
+++ b/package/erlang/Config.in
@@ -1,9 +1,16 @@
+config BR2_PACKAGE_HOST_ERLANG_ARCH_SUPPORTS
+	bool
+	default y if BR2_HOSTARCH = "x86_64"
+	default y if BR2_HOSTARCH = "x86"
+
 config BR2_PACKAGE_ERLANG_ARCH_SUPPORTS
 	bool
 	# see HOWTO/INSTALL.md for Erlang's supported platforms
 	# when using its native atomic ops implementation
 	default y if BR2_i386 || BR2_x86_64 || BR2_powerpc || \
 		BR2_sparc_v9 || BR2_arm || BR2_aarch64 || BR2_mipsel
+	# erlang needs host-erlang
+	depends on BR2_PACKAGE_HOST_ERLANG_ARCH_SUPPORTS
 
 comment "erlang needs a toolchain w/ dynamic library"
 	depends on BR2_USE_MMU # fork()
-- 
2.14.1

[Buildroot] [PATCH] erlang: enable host-erlang for x86/x86_64 architectures only

[Baruch Siach]

Hi Johan,

On Mon, Feb 26, 2018 at 04:59:47PM +0100, Johan Oudinet wrote:
> And make erlang depends on host-erlang.

A reference to the autobuilder failure would be nice.

> Signed-off-by: Johan Oudinet <johan.oudinet@gmail.com>
> ---
>  package/erlang/Config.in | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/package/erlang/Config.in b/package/erlang/Config.in
> index 96af5516fd..bf745555e8 100644
> --- a/package/erlang/Config.in
> +++ b/package/erlang/Config.in
> @@ -1,9 +1,16 @@
> +config BR2_PACKAGE_HOST_ERLANG_ARCH_SUPPORTS
> +	bool
> +	default y if BR2_HOSTARCH = "x86_64"
> +	default y if BR2_HOSTARCH = "x86"
> +
>  config BR2_PACKAGE_ERLANG_ARCH_SUPPORTS
>  	bool
>  	# see HOWTO/INSTALL.md for Erlang's supported platforms
>  	# when using its native atomic ops implementation
>  	default y if BR2_i386 || BR2_x86_64 || BR2_powerpc || \
>  		BR2_sparc_v9 || BR2_arm || BR2_aarch64 || BR2_mipsel
> +	# erlang needs host-erlang
> +	depends on BR2_PACKAGE_HOST_ERLANG_ARCH_SUPPORTS
>  
>  comment "erlang needs a toolchain w/ dynamic library"
>  	depends on BR2_USE_MMU # fork()

This comment also needs the new dependency.

baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -

[Buildroot] [PATCH] erlang: enable host-erlang for x86/x86_64 architectures only

[Johan Oudinet]

On Mon, Feb 26, 2018 at 5:28 PM, Baruch Siach <baruch@tkos.co.il> wrote:
> Hi Johan,
>
> On Mon, Feb 26, 2018 at 04:59:47PM +0100, Johan Oudinet wrote:
>> And make erlang depends on host-erlang.
>
> A reference to the autobuilder failure would be nice.

Thanks Baruch. I'll include the references in the V2.

>> Signed-off-by: Johan Oudinet <johan.oudinet@gmail.com>
>> ---
>>  package/erlang/Config.in | 7 +++++++
>>  1 file changed, 7 insertions(+)
>>
>> diff --git a/package/erlang/Config.in b/package/erlang/Config.in
>> index 96af5516fd..bf745555e8 100644
>> --- a/package/erlang/Config.in
>> +++ b/package/erlang/Config.in
>> @@ -1,9 +1,16 @@
>> +config BR2_PACKAGE_HOST_ERLANG_ARCH_SUPPORTS
>> +     bool
>> +     default y if BR2_HOSTARCH = "x86_64"
>> +     default y if BR2_HOSTARCH = "x86"
>> +
>>  config BR2_PACKAGE_ERLANG_ARCH_SUPPORTS
>>       bool
>>       # see HOWTO/INSTALL.md for Erlang's supported platforms
>>       # when using its native atomic ops implementation
>>       default y if BR2_i386 || BR2_x86_64 || BR2_powerpc || \
>>               BR2_sparc_v9 || BR2_arm || BR2_aarch64 || BR2_mipsel
>> +     # erlang needs host-erlang
>> +     depends on BR2_PACKAGE_HOST_ERLANG_ARCH_SUPPORTS
>>
>>  comment "erlang needs a toolchain w/ dynamic library"
>>       depends on BR2_USE_MMU # fork()
>
> This comment also needs the new dependency.
>

I don't think so.
It already depends on BR2_PACKAGE_ERLANG_ARCH_SUPPORTS, which now
depends on BR2_PACKAGE_HOST_ERLANG_ARCH_SUPPORTS.

-- 
Johan