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.
I'm fine with the changes, thanks for doing it. I think we'll need to add
this option to the yocto-autobuilder-helper config.json entries else
the current testing will take much longer. We're safe there to
assume that dependencies are tested separately.
Right. This is a good point. Let's review the patches first and make sure they work well! I really prefer if it's enabled by default, so that it's more obvious to layers' maintainers if/when there is a problem.
Cheers,
Richard