cederom commented on PR #3120: URL: https://github.com/apache/nuttx-apps/pull/3120#issuecomment-3049681459
> > > @xiaoxiang781216: Before merged, this patch should separate xedge to independent project and split one patch to two: one is xedge project and another is xedge demo. > > > > > > @JorgeGzm: Thank you for the feedback. I understand now that the project should be split into two separate patches: > > ``` > > * Xedge Library - The core toolkit as an independent, reusable component in **apps/xxx/xedge/** > > > > * Xedge Example - A simple demonstration of library usage in **apps/examples/xedgedemo/** > > ``` > > > > Before proceeding with the implementation, I have a question about the proper location for the Xedge library. Would **apps/netutils/xedge/** be the appropriate directory, or would you recommend a different location such as **apps/framework/xedge/** ? > > @xiaoxiang781216: netutils or system is a good candidate. @cederom what's your opinion? Thanks for asking @xiaoxiang781216 :-) I would go for `netutils` as `xedge` is a framework tightly coupled with networking / IoT capabilities while `system` is more about "local" system utilities :-) -- 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
