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