diff mbox series

[v2,2/3] classes/insane: clean up do_qa_sysroot

Message ID 20260703100710.2901608-2-ross.burton@arm.com
State New
Headers show
Series [v2,1/3] classes/insane: create one CachedPath instance in qa_check_staged | expand

Commit Message

Ross Burton July 3, 2026, 10:07 a.m. UTC
do_populate_sysroot copies SYSROOT_DIRS into SYSROOT_DESTDIR, so there's
no need to loop over SYSROOT_DIRS when checking files: qa_check_staged
can just recurse down the entire tree.

Signed-off-by: Ross Burton <ross.burton@arm.com>
---
 meta/classes-global/insane.bbclass | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)
diff mbox series

Patch

diff --git a/meta/classes-global/insane.bbclass b/meta/classes-global/insane.bbclass
index 942b6e500d5..1818ce2168d 100644
--- a/meta/classes-global/insane.bbclass
+++ b/meta/classes-global/insane.bbclass
@@ -1225,9 +1225,7 @@  addtask do_package_qa_setscene
 
 python do_qa_sysroot() {
     bb.note("QA checking do_populate_sysroot")
-    sysroot_destdir = d.expand('${SYSROOT_DESTDIR}')
-    for sysroot_dir in d.expand('${SYSROOT_DIRS}').split():
-        qa_check_staged(sysroot_destdir + sysroot_dir, d)
+    qa_check_staged(d.getVar("SYSROOT_DESTDIR"), d)
     oe.qa.exit_with_message_if_errors("do_populate_sysroot for this recipe installed files with QA issues", d)
 }
 do_populate_sysroot[postfuncs] += "do_qa_sysroot"