btashton commented on a change in pull request #566:
URL: 
https://github.com/apache/incubator-nuttx-apps/pull/566#discussion_r563499356



##########
File path: crypto/mbedtls/Kconfig
##########
@@ -0,0 +1,80 @@
+#
+# For a description of the syntax of this configuration file,
+# see the file kconfig-language.txt in the NuttX tools repository.
+#
+
+menuconfig CRYPTO_MBEDTLS
+       bool "Mbed TLS Cryptography Library"
+       default n
+       ---help---
+               Enable support for Mbed TLS.
+
+if CRYPTO_MBEDTLS
+
+config MBEDTLS_VERSION
+       string "MBEDTLS Version"
+       default "2.25.0"
+
+menuconfig MBEDTLS_APPS

Review comment:
       This seems wrong. The applications and the library are two separate 
things. Enabling the library should have no impact (besides dependency) on the 
applications. 




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to