diff mbox series

[1/2] bitbake.conf: add an addfragments directive for oe-core and dependent layers

Message ID 20241114111100.2624737-1-alex.kanavin@gmail.com
State New
Headers show
Series [1/2] bitbake.conf: add an addfragments directive for oe-core and dependent layers | expand

Commit Message

Alexander Kanavin Nov. 14, 2024, 11:10 a.m. UTC
From: Alexander Kanavin <alex@linutronix.de>

Please see the patch to bitbake for syntax and implementation details.

The path prefix to fragments is in its own variable so it doesn't have
to be hardcoded into tools.

Signed-off-by: Alexander Kanavin <alex@linutronix.de>
---
 meta/conf/bitbake.conf | 2 ++
 1 file changed, 2 insertions(+)

Comments

Mathieu Dubois-Briand Nov. 15, 2024, 12:44 p.m. UTC | #1
On Thu, Nov 14, 2024 at 12:10:59PM +0100, Alexander Kanavin via lists.openembedded.org wrote:
> From: Alexander Kanavin <alex@linutronix.de>
> 
> Please see the patch to bitbake for syntax and implementation details.
> 
> The path prefix to fragments is in its own variable so it doesn't have
> to be hardcoded into tools.
> 
> Signed-off-by: Alexander Kanavin <alex@linutronix.de>
> ---

Hi Alexander,

I'm not sure what is the issue, but it looks like this patch is breaking
the patchtest-selftest on the autobuilder: the remove layers step is
timing out.

https://valkyrie.yoctoproject.org/#/builders/71/builds/431

Can you have a look at this issue ?
Alexander Kanavin Nov. 15, 2024, 2:52 p.m. UTC | #2
On Fri, 15 Nov 2024 at 13:44, Mathieu Dubois-Briand
<mathieu.dubois-briand@bootlin.com> wrote:
> I'm not sure what is the issue, but it looks like this patch is breaking
> the patchtest-selftest on the autobuilder: the remove layers step is
> timing out.
>
> https://valkyrie.yoctoproject.org/#/builders/71/builds/431
>
> Can you have a look at this issue ?

I'll do that; I've also realized the v2 patchset is missing a couple
things I was going to implement (prefixing fragment names with layer
names; using standard library functions for manipulating configs), so
I'll send a v3 and check it with patchtest-selftest first.

Alex
diff mbox series

Patch

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 02bbf0e7a52..b6013f66e26 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -836,6 +836,8 @@  include conf/documentation.conf
 include conf/licenses.conf
 require conf/sanity.conf
 include conf/bblock.conf
+OE_FRAGMENTS_PREFIX ?= "conf/fragments"
+addfragments ${OE_FRAGMENTS_PREFIX} OE_FRAGMENTS
 
 ##################################################################
 # Weak variables (usually to retain backwards compatibility)