You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/03/15 18:50:35 UTC

[GitHub] [incubator-nuttx-apps] mogenson commented on a change in pull request #1075: add Lua interpreter

mogenson commented on a change in pull request #1075:
URL: https://github.com/apache/incubator-nuttx-apps/pull/1075#discussion_r827310447



##########
File path: interpreters/lua/Kconfig
##########
@@ -0,0 +1,90 @@
+#
+# For a description of the syntax of this configuration file,
+# see the file kconfig-language.txt in the NuttX tools repository.
+#
+
+menuconfig INTERPRETERS_LUA
+	bool "Lua interpreter support"
+	default n
+	select ARCH_SETJMP_H
+	select LIBC_FLOATINGPOINT
+	select LIBC_LOCALE
+	select LIBM

Review comment:
       Since Lua needs libm to build, how about adding `depends on LIBM || ARCH_MATH_H`?




-- 
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: commits-unsubscribe@nuttx.apache.org

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