public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
* [PATCH] meta*: Add LAYERSERIES_COMPAT and LAYERSERIES_CORENAMES markup to layer.conf
@ 2017-06-07 12:39 Richard Purdie
  2017-06-07 16:51 ` Leonardo Sandoval
  0 siblings, 1 reply; 3+ messages in thread
From: Richard Purdie @ 2017-06-07 12:39 UTC (permalink / raw)
  To: openembedded-core

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"
 
 # Set a variable to get to the top of the metadata location
-- 
2.7.4



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] meta*: Add LAYERSERIES_COMPAT and LAYERSERIES_CORENAMES markup to layer.conf
  2017-06-07 12:39 [PATCH] meta*: Add LAYERSERIES_COMPAT and LAYERSERIES_CORENAMES markup to layer.conf Richard Purdie
@ 2017-06-07 16:51 ` Leonardo Sandoval
  2017-06-07 17:08   ` Peter Kjellerstedt
  0 siblings, 1 reply; 3+ messages in thread
From: Leonardo Sandoval @ 2017-06-07 16:51 UTC (permalink / raw)
  To: Richard Purdie; +Cc: openembedded-core

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.

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




^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] meta*: Add LAYERSERIES_COMPAT and LAYERSERIES_CORENAMES markup to layer.conf
  2017-06-07 16:51 ` Leonardo Sandoval
@ 2017-06-07 17:08   ` Peter Kjellerstedt
  0 siblings, 0 replies; 3+ messages in thread
From: Peter Kjellerstedt @ 2017-06-07 17:08 UTC (permalink / raw)
  To: Leonardo Sandoval, Richard Purdie
  Cc: openembedded-core@lists.openembedded.org

> -----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



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-06-07 17:08 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-06-07 12:39 [PATCH] meta*: Add LAYERSERIES_COMPAT and LAYERSERIES_CORENAMES markup to layer.conf Richard Purdie
2017-06-07 16:51 ` Leonardo Sandoval
2017-06-07 17:08   ` Peter Kjellerstedt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox