You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by xi...@apache.org on 2021/04/15 13:09:59 UTC

[incubator-nuttx-apps] branch master updated: interpreters/ficl: Fix compilation issue

This is an automated email from the ASF dual-hosted git repository.

xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-apps.git


The following commit(s) were added to refs/heads/master by this push:
     new 496393a  interpreters/ficl: Fix compilation issue
496393a is described below

commit 496393a87b89b6361a0f22b93ef9739b7f555ea9
Author: Alan C. Assis <ac...@gmail.com>
AuthorDate: Tue Apr 13 11:16:52 2021 -0300

    interpreters/ficl: Fix compilation issue
    
    A NuttX user told that ficl wasn't working as suggested in the
    README.txt, then I found that the root cause was the order of the
    Make.srcs inclusion inside the Makefile. I think we can do better:
    we could automate the process to download and configure ficl.
---
 interpreters/ficl/Makefile | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/interpreters/ficl/Makefile b/interpreters/ficl/Makefile
index c3991af..d82c602 100644
--- a/interpreters/ficl/Makefile
+++ b/interpreters/ficl/Makefile
@@ -39,6 +39,10 @@ include $(APPDIR)/Make.defs
 
 # Tools
 
+# Include the generated Make.srcs
+
+-include Make.srcs
+
 # Include paths
 
 CFLAGS += ${shell $(INCDIR) "$(CC)" $(BUILDDIR)/$(FICL_SUBDIR) $(BUILDDIR)/src}
@@ -47,8 +51,6 @@ CFLAGS += ${shell $(INCDIR) "$(CC)" $(BUILDDIR)/$(FICL_SUBDIR) $(BUILDDIR)/src}
 
 CSRCS = nuttx.c
 
--include Make.srcs
-
 ASRCS += $(FICL_ASRCS)
 CXXSRCS += $(FICL_CXXSRCS)
 CSRCS += $(FICL_CSRCS)