You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by pk...@apache.org on 2023/06/22 20:27:33 UTC
[nuttx-apps] 02/02: build: Replace "$(shell $(DEFINE) $(CC) ...)" with $(DEFINE_PREFIX)
This is an automated email from the ASF dual-hosted git repository.
pkarashchenko pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx-apps.git
commit a2f01f5e6465b80efb046918bf7a13c3f0f2374f
Author: Xiang Xiao <xi...@xiaomi.com>
AuthorDate: Thu Jun 22 23:50:01 2023 +0800
build: Replace "$(shell $(DEFINE) $(CC) ...)" with $(DEFINE_PREFIX)
to unify the way to define macros in Makefile
Signed-off-by: Xiang Xiao <xi...@xiaomi.com>
---
Application.mk | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Application.mk b/Application.mk
index 121532cdb..6bf6e6667 100644
--- a/Application.mk
+++ b/Application.mk
@@ -205,8 +205,8 @@ MAINNAME := $(addsuffix _main,$(PROGNAME))
$(MAINCXXOBJ): %$(CXXEXT)$(SUFFIX)$(OBJEXT): %$(CXXEXT)
$(eval MAIN=$(word $(call GETINDEX,$<,$(MAINCXXSRCS)),$(MAINNAME)))
- $(eval $<_CXXFLAGS += ${shell $(DEFINE) "$(CXX)" main=$(MAIN)})
- $(eval $<_CXXELFFLAGS += ${shell $(DEFINE) "$(CXX)" main=$(MAIN)})
+ $(eval $<_CXXFLAGS += ${DEFINE_PREFIX}main=$(MAIN))
+ $(eval $<_CXXELFFLAGS += ${DEFINE_PREFIX}main=$(MAIN))
$(if $(and $(CONFIG_BUILD_LOADABLE),$(CXXELFFLAGS)), \
$(call ELFCOMPILEXX, $<, $@), $(call COMPILEXX, $<, $@))