[Buildroot] [PATCH v1 2/2] package/batman-adv: enable B.A.T.M.A.N. V protocol by default
ps.report at gmx.net
Mon Mar 15 22:12:50 UTC 2021
On Mon, 15 Mar 2021 20:59:00 +0100, "Yann E. MORIN" <yann.morin.1998 at free.fr> wrote:
> Peter, All,
> On 2021-03-14 23:32 +0100, Peter Seiderer spake thusly:
> > As upstream enables CONFIG_BATMAN_ADV_BATMAN_V option since
> > 2018.2 by defaullt () honor this in buildroot and do
> > the same (and remove the leagcy experimental comment accordingly).
> >  https://git.open-mesh.org/batman-adv.git/blob/HEAD:/CHANGELOG.rst#l174
> > Signed-off-by: Peter Seiderer <ps.report at gmx.net>
> > ---
> > package/batman-adv/Config.in | 5 ++---
> > 1 file changed, 2 insertions(+), 3 deletions(-)
> > diff --git a/package/batman-adv/Config.in b/package/batman-adv/Config.in
> > index 9d3d7967b9..f3ac974296 100644
> > --- a/package/batman-adv/Config.in
> > +++ b/package/batman-adv/Config.in
> > @@ -25,11 +25,10 @@ config BR2_PACKAGE_BATMAN_ADV_NC
> > https://www.open-mesh.org/projects/batman-adv/wiki/NetworkCoding
> > config BR2_PACKAGE_BATMAN_ADV_BATMAN_V
> > - bool "enable B.A.T.M.A.N. V protocol (experimental)"
> > + bool "enable B.A.T.M.A.N. V protocol"
> Upstream seems to still consider this to be experimental .
Would count this as a oversight/leftover...and after 3 years of
default to enabled would not count it as experimental...
> > + default y
> What we talked about earlier , was to always enable it, not to change
> the default.
> Now, people that have a defconfig file with BATMAN-V disabled (i.e.
> there would be nothing in the defconfig as 'is not set' is the default),
> will have it enabled now.
I believe the count of people who have BATMAN-V disabled is zero (or they
are to shy to report the compile failure which should occur since the
introduction of BATMAN-V as the ifdef pattern did not change since 2016,
> So, if upstream recommends to not disable it (do you have a ref?), then
> we should just drop our option, and to always enable it.
Same rationale as Christian in the follow-up mail...
>  https://git.open-mesh.org/batman-adv.git/blob/HEAD:/Makefile#l7
>  http://lists.busybox.net/pipermail/buildroot/2021-March/305811.html
> Yann E. MORIN.
> > help
> > Enables B.A.T.M.A.N. V network protocol.
> > - This is the next-gen protocol and is experimental at this
> > - time.
> > https://www.open-mesh.org/projects/batman-adv/wiki/BATMAN_V
> > --
> > 2.30.1
> > _______________________________________________
> > buildroot mailing list
> > buildroot at busybox.net
> > http://lists.busybox.net/mailman/listinfo/buildroot
More information about the buildroot