[Buildroot] [PATCH] package/libcurl: add option for no SSL/TLS support

Yann E. MORIN yann.morin.1998 at free.fr
Tue Dec 31 16:40:23 UTC 2019


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 at free.fr>
Cc: Baruch Siach <baruch at tkos.co.il>
Cc: Matt Weber <matthew.weber at rockwellcollins.com>
Cc: Julien Grossholtz <julien.grossholtz at openest.io>
---
 package/libcurl/Config.in | 15 +++------------
 1 file changed, 3 insertions(+), 12 deletions(-)

diff --git a/package/libcurl/Config.in b/package/libcurl/Config.in
index 3a9ee0f121..652285c0f8 100644
--- a/package/libcurl/Config.in
+++ b/package/libcurl/Config.in
@@ -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
 
+config BR2_PACKAGE_LIBCURL_TLS_NONE
+	bool "None"
+
 endchoice
 
-comment "A TLS library is needed for SSL/TLS support"
-	depends on !BR2_PACKAGE_LIBCURL_TLS_SUPPORT
-
 endif
-- 
2.20.1



More information about the buildroot mailing list