lulingar commented on a change in pull request #3836:
URL: https://github.com/apache/incubator-nuttx/pull/3836#discussion_r650046105



##########
File path: boards/sim/sim/sim/scripts/Make.defs
##########
@@ -83,7 +83,7 @@ endif
 CC = $(CROSSDEV)cc
 CXX = $(CROSSDEV)c++
 CPP = $(CROSSDEV)cc -E -P -x c
-LD = $(CROSSDEV)ld
+LD = $(CROSSDEV)gcc

Review comment:
       Thanks, it failed further down the build, like:
   
       echo "LD:  nuttx"
       LD:  nuttx
       cc -r  -L"/nuttx/staging" -L board  -o nuttx.rel up_head.o --start-group 
-lsched -ldrivers -lboards -lc -lmm -larch -lapps -lnet -lfs -lbinfmt -lboard  
--end-group
       clang: error: unsupported option '--start-group'
       clang: error: unsupported option '--end-group'
       make[1]: Leaving directory '/nuttx/arch/sim/src'
       make[1]: *** [Makefile:314: nuttx] Error 1
       make: *** [tools/Makefile.unix:422: nuttx] Error 2
   
   But it seems #3904 fixes this, I'm testing it out.

##########
File path: boards/sim/sim/sim/scripts/Make.defs
##########
@@ -83,7 +83,7 @@ endif
 CC = $(CROSSDEV)cc
 CXX = $(CROSSDEV)c++
 CPP = $(CROSSDEV)cc -E -P -x c
-LD = $(CROSSDEV)ld
+LD = $(CROSSDEV)gcc

Review comment:
       It didn't fix it, in our clang-only build we don't have gcc. I'm testing 
some other ways to go about this. But this begs the question, why forcing this 
way the compiler to be gcc?




-- 
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


Reply via email to