On Fri, May 22, 2020 at 10:14 PM Xiang Xiao <xiaoxiang781...@gmail.com> wrote: > > On Fri, May 22, 2020 at 8:41 PM Alan Carvalho de Assis > <acas...@gmail.com> wrote: > > > > Hi Xiang, > > > > On 5/22/20, Xiang Xiao <xiaoxiang781...@gmail.com> wrote: > > sic > > > > > > But mbedtls can be used in more context than HTTPS/TLS like security > > > boot, OTA and TEE, it doesn't make sense to put into netutils. A > > > central folder(e.g. external) for 3rd party is a better choice > > > because: > > > > The external folder is used for other purpose (to test user own code). > > Not it is even in the .gitignore. > > And since the code will be there already, it is better to keep the way > > NuttX already to it for external application. > > > > We don't need put Makefile/Kconfig into apps/external directly. Like > Chao's suggestion, we can create a new apache-nuttx-external git to > manage the porting. People can link apps/external to > apache-nuttx-external or their own version.
i guess a user often want to use apache-nuttx-external AND his own version. while you can always workaround it with symbolic links, i wonder if it could be more straightforward to use multiple "external" repos. > > > > 1.New user can find what already provide by NuttX quickly before start > > > porting > > > > Normally new users will look inside menuconfig before looking the > > code, so it is not an appealing reason. > > > > Even with menuconfig, user need dig into several level to find, e.g.: > Application Configuration->Grahpics Support->Littlev Graphic Libray(LVGL) > Considering we have more than 1000 Kconfig options, the new user is > hard to find the library he/she want from Kconfig quickly. > > > > 2.Help PMC check LICENSE file reflect the truth > > > > It doesn't better since the code is not distributed inside NuttX. > > > > > 3.Follow other project practice > > > > > > > Hmm, so I think I didn't get it correctly from start. Maybe your idea > > is to keep the complete code inside an apps/external or > > apps/thirdparty folder, is it? > > No, just Makefile/Kconfig, but we need a system way to layout 3rd > party library if you consider how to find them quickly after we port > 100+ libraries which very likely happen after one or two years. > > > If this is the idea, maybe in this case other option is to have an > > apps-external or thirdparty repository to avoid the apps/ repository > > becoming too big with code from external projects. > > > > It's very important to have a common place to share the 3rd party > library support. Many user select one RTOS just because it support the > library he/she want to use in box. We need catch up here to compete > with other RTOS. > > > >> > 4. how do you think about adding tls support to netutils/webclient? > > >> > > > >> > > >> I think it is better to create the mbedtls as a separated "library" > > >> (note the quotes) instead of mixing it directly inside webclient, > > >> because it could make it easier to users to use mbedtls on their web > > >> applications. But, of course, it should be nice to have an option to > > >> compile the webclient with the mbedtls "library" support. There are > > >> some examples of "libraries" and applications on NuttX apps, i.e.: > > >> gpsutils/minmea/minmea.c and an application using it: examples/gps. > > >> > > > > > > Yes, mbedtls is better to integrate as a library, and then any builtin > > > or 3rd party apps can utilize it. > > > > > > > I suppose this is the way you did at Xiaomi, right? > > > > BR, > > > > Alan
