On Tue, 4 Feb 2025 at 00:08, Raymond Mao <raymond....@linaro.org> wrote:
>
> Fixed the building failures when WGET_HTTPS,NET_LWIP and MBEDTLS_LIB
> are selected due to a few incorrect kconfig dependencies.
>
> Signed-off-by: Raymond Mao <raymond....@linaro.org>
> ---
> Changes in v2:
> - Remove the renaming to patch 3/3 and update the commit message accordingly.
>
>  cmd/Kconfig         | 1 -
>  lib/mbedtls/Kconfig | 3 ++-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/cmd/Kconfig b/cmd/Kconfig
> index e62817bd977..2e2dd459f56 100644
> --- a/cmd/Kconfig
> +++ b/cmd/Kconfig
> @@ -2160,7 +2160,6 @@ config WGET_HTTPS
>         select ASYMMETRIC_PUBLIC_KEY_SUBTYPE
>         select X509_CERTIFICATE_PARSER
>         select PKCS7_MESSAGE_PARSER
> -       select MBEDTLS_LIB_CRYPTO
>         select MBEDTLS_LIB_TLS
>         select RSA_VERIFY_WITH_PKEY
>         select X509_CERTIFICATE_PARSER
> diff --git a/lib/mbedtls/Kconfig b/lib/mbedtls/Kconfig
> index 35d8c507a89..bb8ac5f862b 100644
> --- a/lib/mbedtls/Kconfig
> +++ b/lib/mbedtls/Kconfig
> @@ -156,6 +156,7 @@ endif # LEGACY_CRYPTO_BASIC
>
>  config LEGACY_CRYPTO_CERT
>         bool "legacy certificate libraries"
> +       depends on LEGACY_CRYPTO
>         select ASN1_DECODER_LEGACY if ASN1_DECODER
>         select ASYMMETRIC_PUBLIC_KEY_LEGACY if \
>                 ASYMMETRIC_PUBLIC_KEY_SUBTYPE
> @@ -491,7 +492,7 @@ config MBEDTLS_LIB_TLS
>         depends on ASYMMETRIC_PUBLIC_KEY_MBEDTLS
>         depends on ASN1_DECODER_MBEDTLS
>         depends on ASYMMETRIC_PUBLIC_KEY_MBEDTLS
> -       depends on MBEDTLS_LIB_CRYPTO
> +       depends on MBEDTLS_LIB
>         help
>           Enable MbedTLS TLS library. Required for HTTPs support
>           in wget
> --
> 2.25.1
>

Reviewed-by: Ilias Apalodimas <ilias.apalodi...@linaro.org>

Reply via email to