summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGratian Crisan <gratian.crisan@ni.com>2020-12-04 13:30:21 -0600
committerRichard Purdie <richard.purdie@linuxfoundation.org>2020-12-09 12:20:33 +0000
commit2dcbc66e17794a79b9d2117eb866a3b548f177de (patch)
treec7ba8978b5993a8ee7fa1e2e0f7ca19c0094fe4c
parentd8676e9a0e07ad67aeda21976e0fe0848f078eb7 (diff)
downloadpoky-2dcbc66e17794a79b9d2117eb866a3b548f177de.tar.gz
kernel-module-split.bbclass: fix kernel modules getting marked as CONFFILES
Yi pointed out that commit 1a70a92d1f10 ("kernel-module-split.bbclass: identify kernel modconf files as configuration files") is unintentionally adding the actual kernel /lib/modules .ko files to the CONFFILES variable. The root cause is the re-use of the 'files' variable in that commit. Fix it by using a separate variable to keep track of the generated module .conf files that need to be marked as configuration files. Fixes: 1a70a92d1f10 ("kernel-module-split.bbclass: identify kernel modconf files as configuration files") Reported-by: Yi Zhao <yi.zhao@windriver.com> (From OE-Core rev: db5f2ca532db4f0d2e05b7cb5f9d146e1dd76ab3) Signed-off-by: Gratian Crisan <gratian.crisan@ni.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r--meta/classes/kernel-module-split.bbclass5
1 files changed, 4 insertions, 1 deletions
diff --git a/meta/classes/kernel-module-split.bbclass b/meta/classes/kernel-module-split.bbclass
index c8ede26996..baa32e0a90 100644
--- a/meta/classes/kernel-module-split.bbclass
+++ b/meta/classes/kernel-module-split.bbclass
@@ -120,7 +120,10 @@ python split_kernel_module_packages () {
120 files = d.getVar('FILES_%s' % pkg) 120 files = d.getVar('FILES_%s' % pkg)
121 files = "%s /etc/modules-load.d/%s.conf /etc/modprobe.d/%s.conf" % (files, basename, basename) 121 files = "%s /etc/modules-load.d/%s.conf /etc/modprobe.d/%s.conf" % (files, basename, basename)
122 d.setVar('FILES_%s' % pkg, files) 122 d.setVar('FILES_%s' % pkg, files)
123 d.setVar('CONFFILES_%s' % pkg, files) 123
124 conffiles = d.getVar('CONFFILES_%s' % pkg)
125 conffiles = "%s /etc/modules-load.d/%s.conf /etc/modprobe.d/%s.conf" % (conffiles, basename, basename)
126 d.setVar('CONFFILES_%s' % pkg, conffiles)
124 127
125 if "description" in vals: 128 if "description" in vals:
126 old_desc = d.getVar('DESCRIPTION_' + pkg) or "" 129 old_desc = d.getVar('DESCRIPTION_' + pkg) or ""