SPRESENSE commented on a change in pull request #346:
URL:
https://github.com/apache/incubator-nuttx-apps/pull/346#discussion_r471394813
##########
File path: examples/wget/Kconfig
##########
@@ -12,11 +12,18 @@ config EXAMPLES_WGET
if EXAMPLES_WGET
+config EXAMPLES_WGET_STACKSIZE
+ int "wget stack size"
+ default 2048
+ ---help---
+ Stack size of "wget" command. If you use SSL connection,
+ you should increase this value around 8096.
+
config EXAMPLES_WGET_URL
Review comment:
Sorry, my previous comment was mistake.
I know this patch is wrong.
I think it has 2 ways for fix it. 1) completely remove EXAMPLES_WGET_URL
option and 2) to be able to use both of command line and option.
Do you think which better ones?
----------------------------------------------------------------
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
