Hi Marko, Not a problem. I re-added smp_uart and fixed the build. Thanks.
On Thu, Oct 10, 2019 at 1:30 AM marko kiiskila <ma...@runtime.io> wrote: > Sorry for being difficult here, but.. > > To be exact nmgr_uart came after newtmgr-over-shell. The reason > it came to be was smaller memory footprint platforms, and assumption > was that folks might still want option to newtmgr-over-uart after their > dev was done. > It got picked up for serial bootloader initially as it was small in size, > and easy to integrate. > > I’m going to vote -1 for removing it. I believe the use cases justifying > it’s initial existence are still valid. > > > On 10 Oct 2019, at 0.18, Vipul Rahane <vrah...@gmail.com> wrote: > > > > Thanks Will and Chris for your response, removing it and transitioning it > > to smp_shell. This will be part of the MCUmgr changes to mynewt-core. > > > > > > > > On Wed, Oct 9, 2019 at 1:45 PM will sanfilippo <wi...@runtime.io> wrote: > > > >> +1 on my part but not sure I am in the best position to vote :-) > >> > >>> On Oct 9, 2019, at 12:42 PM, Vipul Rahane <vrah...@gmail.com> wrote: > >>> > >>> Hello, > >>> > >>> While making the changes for MCUmgr, we came across nmgr_uart which is > a > >>> predecessor of nmgr_shell. So far, from what I gather, the > functionality > >> is > >>> the same except for the fact that shell can bring in other code which > can > >>> increase the code size a bit. > >>> > >>> As a solution I was suggesting removing nmgr_uart and transitioning to > >>> smp_shell as part of the MCUmgr changes. > >>> > >>> What does the community think about it and does anybody have issues > with > >> it. > >>> > >>> This question was raised as the CI did not catch errors with smp_uart > and > >>> nothing really uses it in the mynewt ecosystem. I am looking for .a > quick > >>> turn around on this question, so, any input would be fine. Thanks. > >>> > >>> Regards, > >>> Vipul Rahane > >>> > >>> On Fri, Sep 27, 2019 at 11:38 AM Vipul Rahane <vrah...@gmail.com> > wrote: > >>> > >>>> Hello fellow developers :-) > >>>> > >>>> So, Miguel and I have been working on unifying Newtmgr into the MCUmgr > >>>> repo (https://github.com/apache/mynewt-mcumgr). There are no protocol > >>>> level changes, since MCUmgr(SMP) and Newtmgr(NMP) were the same and > so, > >>>> everything should work off the bat for everybody. > >>>> > >>>> The PRs related to this change are: > >>>> 1. https://github.com/apache/mynewt-core/pull/1917 > >>>> 2. https://github.com/apache/mynewt-mcumgr/pull/27 > >>>> 3. https://github.com/apache/mynewt-nimble/pull/577 > >>>> > >>>> We have also added a transient package for backwards compatibility. > So, > >>>> this should be a fairly smooth transition for everybody. > >>>> > >>>> Details about the change: > >>>> > >>>> - Replace newtmgr with SMP and MCUmgr > >>>> - The change addresses issue #1871 > >>>> <https://github.com/apache/mynewt-core/issues/1871> > >>>> - Make stats, logs and img_mgmt work with MCUmgr > >>>> - Make default group commands work with MCUmgr > >>>> - Add transient packages for backwards compatibility > >>>> - Update apps to have the updated dependencies > >>>> - Add syscfgs for MGMT commands Eg: CONFIG_MGMT, STATS_MGMT, > LOG_MGMT, > >>>> etc > >>>> - Make OICMGR use OMP from MCUmgr > >>>> - Transition from Newtmgr -> SMP > >>>> - Remove cborattr and update cborattr in MCUmgr > >>>> - Fix builds for log version 2 > >>>> - We are also removing support for log_fcb_slot1 since it is not > >>>> really used by the community or any modules in the core repo. > >>>> - Updating mynewt port of MCUmgr > >>>> - Adding Stats, Logs, datetime and other default commands support to > >>>> MCUmgr > >>>> - Changing mgmt_group_register() in MCUmgr so that it allows > >>>> re-registration of command groups > >>>> - Fix dependency and build issues in MCUmgr for mynewt port and apps > >>>> - Adding support for log version 3 and image hash changes > >>>> - Adding support for image management lazy erase > >>>> - Adding support for config mgmt > >>>> - Updating cborattr in MCUmgr to have similar support as mynewt-core > >>>> - Fix unit tests and add some more for cborattr just like in > >>>> mynewt-core > >>>> - Change log_mgmt to use fragmenting so that an entry can be divided > >>>> into smaller chunks while reading. > >>>> - Fix mynewt port of os_mgmt > >>>> - Add changes for backwards compatibility > >>>> - Add sample app: omp_svr for trying out OICMGR over BLE/Serial > >>>> - Fix reboot_log for soft resets > >>>> > >>>> > >>>> Some more stuff that would follow after this change would be: > >>>> 1. Release version for MCUmgr > >>>> 2. Unifying MCUmgr tool > >>>> > >>>> Also, there were a lot of fixes that got in as part of this effort in > >> the > >>>> MCUmgr repo. I do encourage everybody to try it out. > >>>> > >>>> -- > >>>> > >>>> Regards, > >>>> Vipul Rahane > >>>> > >>> > >>> > >>> -- > >>> > >>> Regards, > >>> Vipul Rahane > >> > >> > > > > -- > > > > Regards, > > Vipul Rahane > > -- Regards, Vipul Rahane