I assume this patch is for Denzil? It seems that the current OE-Core master has etc in that list.

Please confirm

Sau!

On 09/12/2012 06:41 AM, Mark Asselstine wrote:
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 <mark.asselst...@windriver.com>
CC: Bruce Ashfield <bruce.ashfi...@windriver.com>
---

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)



_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Reply via email to