package/libcurl: add option for no SSL/TLS support
authorYann E. MORIN <yann.morin.1998@free.fr>
Tue, 31 Dec 2019 16:40:23 +0000 (17:40 +0100)
committerThomas Petazzoni <thomas.petazzoni@bootlin.com>
Tue, 31 Dec 2019 17:11:45 +0000 (18:11 +0100)
Since we already have a choice to select the backend to do crypto, push
the limits even further and add an option to do no crypto.

Usually, we would have added that option first in the choice, but if we
were to do that now, existing defconfigs that previously used openssl
(the first item in the choice) would now default to non crypto, which is
not so nice. So we add the new option last in the choice.

Each crypto backend option is used in a conditional block, each of which
default to disabling said backend. So, selecting none will indeed
disable all.

We can now drop the blind intermediate option that would hide the choice
when no backend library was available; there will now always be at least
the none option in the choice, so we need not hide it.

Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
Cc: Baruch Siach <baruch@tkos.co.il>
Cc: Matt Weber <matthew.weber@rockwellcollins.com>
Cc: Julien Grossholtz <julien.grossholtz@openest.io>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
package/libcurl/Config.in

index 3a9ee0f1213866f0e69317368baf97a6ad38eeff..652285c0f863c3cf61ddcf7f06209dd4e474901d 100644 (file)
@@ -45,17 +45,8 @@ config BR2_PACKAGE_LIBCURL_EXTRA_PROTOCOLS_FEATURES
          - DICT
          - Gopher
 
-config BR2_PACKAGE_LIBCURL_TLS_SUPPORT
-       bool
-       default y if BR2_PACKAGE_OPENSSL
-       default y if BR2_PACKAGE_GNUTLS
-       default y if BR2_PACKAGE_LIBNSS
-       default y if BR2_PACKAGE_MBEDTLS
-       default y if BR2_PACKAGE_WOLFSSL
-
 choice
        prompt "SSL/TLS library to use"
-       depends on BR2_PACKAGE_LIBCURL_TLS_SUPPORT
 
 config BR2_PACKAGE_LIBCURL_OPENSSL
        bool "OpenSSL"
@@ -77,9 +68,9 @@ config BR2_PACKAGE_LIBCURL_WOLFSSL
        bool "WolfSSL"
        depends on BR2_PACKAGE_WOLFSSL
 
-endchoice
+config BR2_PACKAGE_LIBCURL_TLS_NONE
+       bool "None"
 
-comment "A TLS library is needed for SSL/TLS support"
-       depends on !BR2_PACKAGE_LIBCURL_TLS_SUPPORT
+endchoice
 
 endif