Re: [PATCH] meta*: Add LAYERSERIES_COMPAT and LAYERSERIES_CORENAMES markup to layer.conf

[Peter Kjellerstedt <peter.kjellerstedt@axis.com>]

> -----Original Message-----
> From: openembedded-core-bounces@lists.openembedded.org
> [mailto:openembedded-core-bounces@lists.openembedded.org] On Behalf Of
> Leonardo Sandoval
> Sent: den 7 juni 2017 18:51
> To: Richard Purdie <richard.purdie@linuxfoundation.org>
> Cc: openembedded-core@lists.openembedded.org
> Subject: Re: [OE-core] [PATCH] meta*: Add LAYERSERIES_COMPAT and
> LAYERSERIES_CORENAMES markup to layer.conf
> 
> On Wed, 2017-06-07 at 13:39 +0100, Richard Purdie wrote:
> > This means mismatched layers are more clearly identified to the user
> in
> > cases where compatibility has not been tested. This is perhaps not as
> > needed for the core repository (other than CORENAMES) but lets lead
> by
> > example.
> >
> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> > ---
> >  meta-selftest/conf/layer.conf | 2 ++
> >  meta-skeleton/conf/layer.conf | 2 ++
> >  meta/conf/layer.conf          | 2 ++
> >  3 files changed, 6 insertions(+)
> >
> > diff --git a/meta-selftest/conf/layer.conf b/meta-
> selftest/conf/layer.conf
> > index a847b78..2a71895 100644
> > --- a/meta-selftest/conf/layer.conf
> > +++ b/meta-selftest/conf/layer.conf
> > @@ -8,3 +8,5 @@ BBFILES += "${LAYERDIR}/recipes-*/*/*.bb \
> >  BBFILE_COLLECTIONS += "selftest"
> >  BBFILE_PATTERN_selftest = "^${LAYERDIR}/"
> >  BBFILE_PRIORITY_selftest = "5"
> > +
> > +LAYERSERIES_COMPAT_selftest = "rocko"
> > diff --git a/meta-skeleton/conf/layer.conf b/meta-
> skeleton/conf/layer.conf
> > index aca1633..a15516a 100644
> > --- a/meta-skeleton/conf/layer.conf
> > +++ b/meta-skeleton/conf/layer.conf
> > @@ -13,3 +13,5 @@ BBFILE_PRIORITY_skeleton = "1"
> >  LAYERVERSION_skeleton = "1"
> >
> >  LAYERDEPENDS_skeleton = "core"
> > +
> > +LAYERSERIES_COMPAT_skeleton = "rocko"
> > diff --git a/meta/conf/layer.conf b/meta/conf/layer.conf
> > index fc16502..6d36aa1 100644
> > --- a/meta/conf/layer.conf
> > +++ b/meta/conf/layer.conf
> > @@ -11,6 +11,8 @@ BBFILE_PRIORITY_core = "5"
> >  # cause compatibility issues with other layers
> >  LAYERVERSION_core = "10"
> >
> > +LAYERSERIES_CORENAMES = "rocko"
> > +
> >  BBLAYERS_LAYERINDEX_NAME_core = "openembedded-core"
> 
> at oe-core, Is there a reason not to set all LAYERSERIES_COMPAT_* =
> "${LAYERSERIES_CORENAMES}"? this would make it easier to bump the
> COMPAT set of variables.

That is probably a bad idea, especially if the aim here is to lead 
by example, since otherwise you risk other layers to just blindly 
copying that and then we have gained nothing...

> >  # Set a variable to get to the top of the metadata location
> > --
> > 2.7.4

//Peter