Yes, please! Brian.
On Fri, Nov 8, 2019 at 10:08 AM will sanfilippo <wi...@runtime.io> wrote: > Fine with me too :-) > > > On Nov 8, 2019, at 8:02 AM, marko kiiskila <ma...@runtime.io> wrote: > > > > > > > >> On 7 Nov 2019, at 16.50, Christopher Collins <ch...@runtime.io> wrote: > >> > >> On Thu, Nov 07, 2019 at 11:56:36AM -0800, Vipul Rahane wrote: > >>> Hello, > >>> > >>> LOG_VERSION 2 has been around for quite some time but is mostly just a > >>> string based log. LOG_VERSION 3 supports string based logs as well as > >>> others. LOG_VERSION 2 does not play well with the MCUmgr mobile > library as > >>> well. > >>> > >>> I would like to suggest getting rid of LOG_VERSION 2 which would allow > us > >>> to make the code a bit simpler and not worrying about backwards > >>> compatibility. > >>> > >>> This discussion was mainly triggered by a PR > >>> https://github.com/apache/mynewt-core/pull/2087 by Jerzy for adding > back > >>> string based reboot log just to make it work with LOG_VERSION 2. > >>> > >>> LOG_VERSION 2 also doesn't play well with the MCUmgr mobile library. > >>> > >>> I suggest we deprecate LOG_VERSION 2 and make the default 3 going > forward. > >> > >> I agree. Version 3 has been around for quite a while (two years?) and > >> it is superior to version 2. I am fine with dropping support for > >> version 2. > >> > > > > I’d ok with that one as well. > >