On Mon, 2013-07-29 at 15:34 +0100, Burton, Ross wrote:
> On 29 July 2013 09:09, Shakeel, Muhammad <muhammad_shak...@mentor.com> wrote:
> > +        # If systemd_unitdir contains anything, delete sysv_initddir
> > +        if (os.path.exists(systemd_unitdir) and 
> > os.listdir(systemd_unitdir)):
> > +            shutil.rmtree(sysv_initddir)
> 
> Not sure we need the listdir(), why should the function only delete
> the directory if it has contents?  If it doesn't have contents you'll
> be packaging an empty directory.

The directory he's deleting is different to the one that he's listing.

p.


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

Reply via email to