[OE-core] [PATCH] kernel: remove unused 'etc' directory
Mark Asselstine
mark.asselstine at windriver.com
Wed Sep 12 19:35:19 UTC 2012
On September 12, 2012 11:20:05 Saul Wold wrote:
> I assume this patch is for Denzil? It seems that the current OE-Core
> master has etc in that list.
>
> Please confirm
>
> Sau!
Saul,
Sorry I got mixed up on my branches there and didn't notice your change,
thanks for straightening me out. Scott any chance we can get Saul's commit
963742f5a9bc [kernel: Fix packaging issue] cherry-picked to Denzil?
Mark
>
> 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.asselstine at windriver.com>
> > CC: Bruce Ashfield <bruce.ashfield at 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)
More information about the Openembedded-core
mailing list