xiaoxiang781216 commented on PR #2239:
URL: https://github.com/apache/nuttx-apps/pull/2239#issuecomment-1868435272

   > @xiaoxiang781216 but popen is not used by default by the project, so there 
is not possibility to it be disabled and break the relation.
   > 
   
   user may call both pipes and popen initially, and remove popen call later, 
and will surprise that linker complain pipes function can't be found after he 
remove CONFIG_SYSTEM_POPEN from defconfig.
   
   > I think "depends on" doesn't protect the feature as we are thinking, 
because the user could go there and enable manually the feature in the .config.
   
   The user will find that the feature doesn't enable actually because the 
dependence doesn't match, instead linker error. Why do you think `depends on` 
doesn't protect the feature?
   
   > And even worst, "depends on" hides the feature in menuconfig.
   >
   > The user will search for it, will find it, but when he goes there the 
option is not there. I think this is a case where "select" is better than 
"depends on"
   
   The user need understand the basic Kconfig concept, I don't believe that 
they can work with NuttX or Zephyr/Linux efficiently if they don't understand 
`depends on` and `select`.
   


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