DORMANTno replies

[PATCH] yocto-check-layer: Only note a layer without a conf/layer.conf (versus error)

From: Richard Purdie <hidden>
Date: 2021-05-06 15:54:32
Subsystem: the rest · Maintainer: Linus Torvalds

From: Bruce Ashfield <redacted>

Dynamic layers may have a conf directory, but don't need (or
want) a conf/layer.conf

This isn't an error, so we can just log it and indicate the
layer is being skipped.

A full layer without a conf file isn't all that useable, so we
aren't letting anything subtle slip through by just logging
it.

Signed-off-by: Bruce Ashfield <redacted>
Signed-off-by: Richard Purdie <redacted>
---
 scripts/yocto-check-layer | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/yocto-check-layer b/scripts/yocto-check-layer
index deba3cb4f8c..44e77b73d0d 100755
--- a/scripts/yocto-check-layer
+++ b/scripts/yocto-check-layer
@@ -112,7 +112,7 @@ def main():
                      % layer['name'])
             layers.remove(layer)
         elif layer['type'] == LayerType.ERROR_NO_LAYER_CONF:
-            logger.error("%s: Don't have conf/layer.conf file."\
+            logger.info("%s: Doesn't have conf/layer.conf file, so ignoring"\
                      % layer['name'])
             layers.remove(layer)
         else:
-- 
2.30.2
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help