There is no real point in adjusting overrides and creating a copy of the datastore, just to access a single variable. We can do this just as easily with a slightly more complicated getVar call. This improves performance.
Signed-off-by: Richard Purdie <richard.pur...@linuxfoundation.org> --- meta/classes/package.bbclass | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass index 51ab009..4e7db9a 100644 --- a/meta/classes/package.bbclass +++ b/meta/classes/package.bbclass @@ -949,18 +949,10 @@ python populate_packages () { seen = [] for pkg in package_list: - localdata = bb.data.createCopy(d) root = os.path.join(pkgdest, pkg) bb.mkdirhier(root) - localdata.setVar('PKG', pkg) - overrides = localdata.getVar('OVERRIDES', True) - if not overrides: - raise bb.build.FuncFailed('OVERRIDES not defined') - localdata.setVar('OVERRIDES', overrides + ':' + pkg) - bb.data.update_data(localdata) - - filesvar = localdata.getVar('FILES', True) or "" + filesvar = d.getVar('FILES_%s' % pkg, True) or d.getVar('FILES', True) or "" if "//" in filesvar: bb.warn("FILES variable for package %s contains '//' which is invalid. Attempting to fix this but you should correct the metadata.\n" % pkg) filesvar.replace("//", "/") @@ -1023,7 +1015,6 @@ python populate_packages () { if ret is False or ret == 0: raise bb.build.FuncFailed("File population failed") - del localdata os.chdir(workdir) unshipped = [] -- 1.7.10.4 _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core