ValentiWorkLearning commented on code in PR #16312:
URL: https://github.com/apache/nuttx/pull/16312#discussion_r2073584351


##########
tools/Unix.mk:
##########
@@ -768,6 +768,7 @@ savedefconfig: apps_preconfig
        $(Q) rm -f warning.tmp
        $(Q) rm -f defconfig.tmp
        $(Q) rm -f sortedconfig.tmp
+       $(info "The defconfig was generated successfully at $(PWD)/defconfig 
.If you're building out-of-tree configuration, then don't forget to copy it 
into your defconfig original location")

Review Comment:
   @anchao hm.. That's good idea. It seems that this flow has to be same for 
other tools(buildroot for instance). So, technically make savedefconfig should 
directly write to the original defconfig file. Would this be the expected 
behaviour? 



-- 
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-unsubscr...@nuttx.apache.org

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

Reply via email to