diff options
| author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2018-07-02 12:43:50 +0000 | 
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2018-07-02 17:13:19 +0100 | 
| commit | 2c1978fe1a5b72167c49010fbdd39a9e2eefdef8 (patch) | |
| tree | 461e884a16b6b802db39a297eee94e39d853a0a2 | |
| parent | af7cd7823a8cc95c9f849498b2416cdb0d4d103e (diff) | |
| download | openembedded-core-2c1978fe1a5b72167c49010fbdd39a9e2eefdef8.tar.gz openembedded-core-2c1978fe1a5b72167c49010fbdd39a9e2eefdef8.tar.bz2 openembedded-core-2c1978fe1a5b72167c49010fbdd39a9e2eefdef8.zip | |
staging: Improve fixup processing code
With the fixes to other parts of multilib, it was found that the fixup code's
assumptions about the recipe sysroot were incorrect. We need to use the value
calculated earlier in the function.
It turns out there is a rather neat way to do this which cleans up the code
as an added bonus.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | meta/classes/staging.bbclass | 18 | 
1 files changed, 6 insertions, 12 deletions
| diff --git a/meta/classes/staging.bbclass b/meta/classes/staging.bbclass index f04c7b637d..91cbc2083d 100644 --- a/meta/classes/staging.bbclass +++ b/meta/classes/staging.bbclass @@ -383,8 +383,6 @@ python extend_recipe_sysroot() {      lock = bb.utils.lockfile(recipesysroot + "/sysroot.lock")      fixme = {} -    fixme[''] = [] -    fixme['native'] = []      seendirs = set()      postinsts = []      multilibs = {} @@ -483,12 +481,13 @@ python extend_recipe_sysroot() {          if manifest:              newmanifest = collections.OrderedDict() +            targetdir = destsysroot              if native: -                fm = fixme['native']                  targetdir = recipesysrootnative -            else: -                fm = fixme[''] -                targetdir = destsysroot +            if targetdir not in fixme: +                fixme[targetdir] = [] +            fm = fixme[targetdir] +              with open(manifest, "r") as f:                  manifests[dep] = manifest                  for l in f: @@ -546,12 +545,7 @@ python extend_recipe_sysroot() {      bb.note("Skipping as already exists in sysroot: %s" % str(msg_exists))      for f in fixme: -        if f == '': -            staging_processfixme(fixme[f], recipesysroot, recipesysroot, recipesysrootnative, d) -        elif f == 'native': -            staging_processfixme(fixme[f], recipesysrootnative, recipesysroot, recipesysrootnative, d) -        else: -            staging_processfixme(fixme[f], multilibs[f].getVar("RECIPE_SYSROOT"), recipesysroot, recipesysrootnative, d) +        staging_processfixme(fixme[f], f, recipesysroot, recipesysrootnative, d)      for p in postinsts:          subprocess.check_output(p, shell=True, stderr=subprocess.STDOUT) | 
