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


##########
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:
   > @acassis please think about when users would want to do the 
**savedefconfig**. Could it be to save a copy? I don't think so, What they 
actually want is to replace the original version.
   
   @anchao I think we are talking about different things. Normally users create 
a new config using an old one as base. But of course, it is possible to change 
the way the do it, now instead of using boardname:nsh as start point the need 
first to copy the "nsh" board profile to a new place (i.e. "newbrdcfg") and the 
run boardname:newbrdcfg
   
   I think this approach is ok, we just need to change the way normally people 
do it (as I described previously)



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