csanchezdll commented on code in PR #3646: URL: https://github.com/apache/nuttx-apps/pull/3646#discussion_r3600701163
########## interpreters/jimtcl/Kconfig: ########## @@ -0,0 +1,12 @@ +# +# For a description of the syntax of this configuration file, +# see the file kconfig-language.txt in the NuttX tools repository. +# + +config INTERPRETERS_JIMTCL + bool "The Jim (TCL) Interpreter" + default n Review Comment: In theory, setjmp support usually comes from the toolchain. That's why by default ARCH_SETJMP is off. Some architectures might be able to use it from the toolchain directly (no need to select ARCH_SETJMP) but others do require it (this is the case for arm-none-eabi I am using, because toolchain setjmp comes from newlib implementation in libc, and it not linked in NuttX). Making a depend here will force to use the NuttX alternative even for architectures where the toolchain provide a working setjmp. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
