diff mbox

[1/1] rng-tools: Fix disabling package on non-x86 architectures.

Message ID 20170111161950.28199-1-nicolas.cavallari@green-communications.fr
State Accepted
Headers show

Commit Message

Nicolas Cavallari Jan. 11, 2017, 4:19 p.m. UTC
Commit 2f89476 ("package/libgpg-error: bump to version 1.23") introduced
a BR2_PACKAGE_LIBGPG_ERROR_ARCH_SUPPORTS which was added as a dependency
to all libgpg-error reverse dependencies.

However, rng-tools only select libgpg-error if BR2_i386 || BR2_x86_64
(for RDRAND) but the rng-tools dependency was added as
BR2_PACKAGE_LIBGPG_ERROR_ARCH_SUPPORTS && (BR2_i386 || BR2_x86_64),
Therefore limiting rng-tools to the x86 architecture.

This patch changes it as ..._ARCH_SUPPORT || !(BR2_i386 || BR2_x86_64).

Signed-off-by: Nicolas Cavallari <nicolas.cavallari@green-communications.fr>
---
 package/rng-tools/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Romain Naour Jan. 22, 2017, 11:53 a.m. UTC | #1
Hi Nicolas,

Le 11/01/2017 à 17:19, Nicolas Cavallari a écrit :
> Commit 2f89476 ("package/libgpg-error: bump to version 1.23") introduced
> a BR2_PACKAGE_LIBGPG_ERROR_ARCH_SUPPORTS which was added as a dependency
> to all libgpg-error reverse dependencies.
> 
> However, rng-tools only select libgpg-error if BR2_i386 || BR2_x86_64
> (for RDRAND) but the rng-tools dependency was added as
> BR2_PACKAGE_LIBGPG_ERROR_ARCH_SUPPORTS && (BR2_i386 || BR2_x86_64),
> Therefore limiting rng-tools to the x86 architecture.
> 
> This patch changes it as ..._ARCH_SUPPORT || !(BR2_i386 || BR2_x86_64).

Reviewed-by: Romain Naour <romain.naour@gmail.com>

Best regards,
Romain

> 
> Signed-off-by: Nicolas Cavallari <nicolas.cavallari@green-communications.fr>
> ---
>  package/rng-tools/Config.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/package/rng-tools/Config.in b/package/rng-tools/Config.in
> index 0c538cc292..b00477d336 100644
> --- a/package/rng-tools/Config.in
> +++ b/package/rng-tools/Config.in
> @@ -1,6 +1,6 @@
>  config BR2_PACKAGE_RNG_TOOLS
>  	bool "rng-tools"
> -	depends on BR2_PACKAGE_LIBGPG_ERROR_ARCH_SUPPORTS && (BR2_i386 || BR2_x86_64) # libgcrypt
> +	depends on BR2_PACKAGE_LIBGPG_ERROR_ARCH_SUPPORTS || !(BR2_i386 || BR2_x86_64) # libgcrypt
>  	select BR2_PACKAGE_ARGP_STANDALONE if BR2_TOOLCHAIN_USES_UCLIBC || BR2_TOOLCHAIN_USES_MUSL
>  	# For rdrand ligcrypt is required and it's not obvious to users
>  	select BR2_PACKAGE_LIBGCRYPT if BR2_i386 || BR2_x86_64
>
Thomas Petazzoni Jan. 24, 2017, 10:20 a.m. UTC | #2
Hello,

On Wed, 11 Jan 2017 17:19:50 +0100, Nicolas Cavallari wrote:
> Commit 2f89476 ("package/libgpg-error: bump to version 1.23") introduced
> a BR2_PACKAGE_LIBGPG_ERROR_ARCH_SUPPORTS which was added as a dependency
> to all libgpg-error reverse dependencies.
> 
> However, rng-tools only select libgpg-error if BR2_i386 || BR2_x86_64
> (for RDRAND) but the rng-tools dependency was added as
> BR2_PACKAGE_LIBGPG_ERROR_ARCH_SUPPORTS && (BR2_i386 || BR2_x86_64),
> Therefore limiting rng-tools to the x86 architecture.
> 
> This patch changes it as ..._ARCH_SUPPORT || !(BR2_i386 || BR2_x86_64).
> 
> Signed-off-by: Nicolas Cavallari <nicolas.cavallari@green-communications.fr>
> ---
>  package/rng-tools/Config.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied to master, thanks.

Thomas
diff mbox

Patch

diff --git a/package/rng-tools/Config.in b/package/rng-tools/Config.in
index 0c538cc292..b00477d336 100644
--- a/package/rng-tools/Config.in
+++ b/package/rng-tools/Config.in
@@ -1,6 +1,6 @@ 
 config BR2_PACKAGE_RNG_TOOLS
 	bool "rng-tools"
-	depends on BR2_PACKAGE_LIBGPG_ERROR_ARCH_SUPPORTS && (BR2_i386 || BR2_x86_64) # libgcrypt
+	depends on BR2_PACKAGE_LIBGPG_ERROR_ARCH_SUPPORTS || !(BR2_i386 || BR2_x86_64) # libgcrypt
 	select BR2_PACKAGE_ARGP_STANDALONE if BR2_TOOLCHAIN_USES_UCLIBC || BR2_TOOLCHAIN_USES_MUSL
 	# For rdrand ligcrypt is required and it's not obvious to users
 	select BR2_PACKAGE_LIBGCRYPT if BR2_i386 || BR2_x86_64