You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by "fjpanag (via GitHub)" <gi...@apache.org> on 2023/03/08 14:02:16 UTC

[GitHub] [nuttx-apps] fjpanag opened a new pull request, #1646: nsh: Added conditional include of netinit.

fjpanag opened a new pull request, #1646:
URL: https://github.com/apache/nuttx-apps/pull/1646

   ## Summary
   
   `netutils/netinit.h` was always included unconditionaly, regardless whether it was needed or not.
   
   This causes some problems with projects without the need of netinit, or networking whatsoever.  
   Sometimes, the apps repo is used partially, netutils may not even be available.
   
   This PR makes sure that this file is included only when needed.
   
   ## Impact
   
   NSH can be build without dependency on netutils.
   
   ## Testing
   
   Build test.
   


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

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


[GitHub] [nuttx-apps] xiaoxiang781216 merged pull request #1646: nsh: Added conditional include of netinit.

Posted by "xiaoxiang781216 (via GitHub)" <gi...@apache.org>.
xiaoxiang781216 merged PR #1646:
URL: https://github.com/apache/nuttx-apps/pull/1646


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

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