From patchwork Wed Sep 12 13:41:00 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: kernel: remove unused 'etc' directory Date: Wed, 12 Sep 2012 13:41:00 -0000 From: Mark Asselstine X-Patchwork-Id: 36427 Message-Id: <1347457260-25574-1-git-send-email-mark.asselstine@windriver.com> To: Along with checks to remove empty modprobe.d and modules-load.d directories we can add a similar check and removal of the etc directory. This will avoid warnings by the sanity checker. Signed-off-by: Mark Asselstine CC: Bruce Ashfield --- There is some concern that removing this directory could cause issues if folks are adding to etc in a bbappend and not checking for and creating this directory. I don't believe this is an issue as if you want to add something to etc it should be done as part of a populate_packages_prepend or earlier since there are sanity checks run as part of populate_packages, so the removal here is being done at the latest possible point in the process (populate_packages_prepend in a bbappend will run before the populate_packages_prepend in kernel.bbclass). meta/classes/kernel.bbclass | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass index 5e04fda..decf6f5 100644 --- a/meta/classes/kernel.bbclass +++ b/meta/classes/kernel.bbclass @@ -468,7 +468,7 @@ python populate_packages_prepend () { # avoid warnings. removedirs only raises an OSError if an empty # directory cannot be removed. dvar = d.getVar('PKGD', True) - for dir in ["%s/etc/modprobe.d" % (dvar), "%s/etc/modules-load.d" % (dvar)]: + for dir in ["%s/etc/modprobe.d" % (dvar), "%s/etc/modules-load.d" % (dvar), "%s/etc/" % (dvar)]: if len(os.listdir(dir)) == 0: os.rmdir(dir)