[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openembedded-core
Subject:    Re: [OE-core] [PATCH 0/4] yocto-check-layer: add support to check for dependencies
From:       "Richard Purdie" <richard.purdie () linuxfoundation ! org>
Date:       2021-07-30 9:27:56
Message-ID: 98ab2870e07ad6d296d29322b7ee78bade43b9ea.camel () linuxfoundation ! org
[Download RAW message or body]

On Fri, 2021-07-30 at 11:08 +0200, Nicolas Dechesne wrote:
> 
> 
> On Fri, Jul 30, 2021 at 10:03 AM Richard Purdie \
> <richard.purdie@linuxfoundation.org> wrote:
> > On Thu, 2021-07-22 at 14:46 +0200, Nicolas Dechesne wrote:
> > > It has been recentely mentioned to me by the YP TSC that a layer must
> > > not only pass the yocto-check-layer script, but also all its
> > > dependencies. These patches add support for testing dependencies
> > > automatically.
> > > 
> > > Patch #1 and #3 are not much related, but improvements I made along
> > > the lines. 
> > > 
> > > Patch #2 split the internal function into 2 parts, so that we can get
> > > the list of dependencies from the main script. 
> > > 
> > > Patch #4 is where the processing of the dependencies is done. I chose
> > > to enable checking the dependencies by default, and use
> > > '--no-auto-dependency' to disable it, we could decide the other way
> > > around. For any dependency of the 'layers under test' we will simply
> > > add them to the list of layers to test unless they are already there.
> > > 
> > > Here is a sample output with these changes:
> > > 
> > > $ yocto-check-layer /work/oe/sources/meta-aws \ 
> > > --dependency /work/oe/sources/meta-openembedded/ 
> > > 
> > > INFO: Summary of results:
> > > INFO:
> > > INFO: meta-aws ... PASS
> > > INFO: meta-python ... PASS
> > > INFO: meta-oe ... PASS
> > > INFO: meta-networking ... PASS
> > 
> > We merged this but have started seeing:
> > 
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/121/builds/166
> > 
> > which seems a little strange and is probably related to this series.
> > 
> 
> 
> yes, it's related.
> 
> ...
> INFO: Adding meta-poky to the list of layers to test, as a dependency
> ...
> INFO: meta-poky ... SKIPPED (Layer under test should not present in BBLAYERS)
> 
> With these changes, we are now adding all dependencies and trying to test them too. \
> But meta-poky seems to be in bblayers.conf, and yocto-check-layer won't work since \
> it needs to compute all checksum *without* the layer in the config first. 
> 
> It is probably a good idea to make sure bblayers only has oe-core when we try to \
> perform YP compat. Alternatively, as discussed, we use --no-auto-dependency on AB.. \
> 

Fair enough, I'll update the config to do that.

Cheers,

Richard

> 



-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#154265): https://lists.openembedded.org/g/openembedded-core/message/154265
Mute This Topic: https://lists.openembedded.org/mt/84377963/4454766
Group Owner: openembedded-core+owner@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [openembedded-core@marc.info]
-=-=-=-=-=-=-=-=-=-=-=-



[prev in list] [next in list] [prev in thread] [next in thread]