[Buildroot] [PATCH] package/libopenssl does not support riscv32

Arnout Vandecappelle arnout at mind.be
Thu Mar 4 19:36:02 UTC 2021


 Hi Yann,

On 04/03/2021 12:14, Yann E. MORIN wrote:

> Drop the default from the choice selection; it was anyway superfluous:
> the default of a choice, if left unspecified, is the first entry of the
> choice.

 Bikeshedding time!

 IMHO, it is still good to have an explicit default. Its effect remains exactly
the same (if the default cannot be selected due to missing dependencies, the
first available options will be chosen), but it makes it clear for the reader
what is the default, and it makes sure that inserting a new option or reordering
the options doesn't affect the default.

[snip]
>  config BR2_PACKAGE_HOSTAPD_WPA3
>  	bool "Enable WPA3 support"
> +	depends on BR2_PACKAGE_LIBOPENSSL_ARCH_SUPPORTS

 We normally add a comment of depends on. But I guess it's pretty obvious here.

>  	select BR2_PACKAGE_OPENSSL
>  	select BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL
>  	help


[snip]
> diff --git a/package/tpm2-tss/Config.in b/package/tpm2-tss/Config.in
> index 2fc56dbd76..b1ab00dd38 100644
> --- a/package/tpm2-tss/Config.in
> +++ b/package/tpm2-tss/Config.in
> @@ -1,6 +1,7 @@
>  config BR2_PACKAGE_TPM2_TSS

 Unfortunately, TPM2_TSS is selected by ima-evm-utils, tpm2-abrmd, tpm2-pkcs11,
tpm2-tools, tpm2-totp. Fortunately, it doesn't recurse any further.

[snip]
>  config BR2_PACKAGE_WPA_SUPPLICANT_MESH_NETWORKING

 WPA_SUPPLICANT is selected by connman but fortunately no mesh or WPA3. We got
lucky there!


 With the propagation of the TPM2_TSS dependency (and for those, a comment
*should* be added), you can add my

 Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>

and apply directly to master.

 Regards,
 Arnout

>  	bool "Enable mesh networking"
>  	depends on BR2_PACKAGE_WPA_SUPPLICANT_AP_SUPPORT
> +	depends on BR2_PACKAGE_LIBOPENSSL_ARCH_SUPPORTS
>  	select BR2_PACKAGE_OPENSSL
>  	select BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL
>  	help
> @@ -83,6 +84,7 @@ config BR2_PACKAGE_WPA_SUPPLICANT_WPS
>  
>  config BR2_PACKAGE_WPA_SUPPLICANT_WPA3
>  	bool "Enable WPA3 support"
> +	depends on BR2_PACKAGE_LIBOPENSSL_ARCH_SUPPORTS
>  	select BR2_PACKAGE_OPENSSL
>  	select BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL
>  	help
> 


More information about the buildroot mailing list