[OE-core] [PATCH] package.bbclass: Reverse runtime symlinks should be tied to package generation

[Lauren Post]

This patch is breaking our imx6slevk builds which do not support gles2.    I can build slevk on its own but not after another machine that supports gles.  I get an error with the runtime-reverse pkgdata for imx6slevk looking for gles2 which is not provided on this machine.
	imx6slevk/pkgdata/runtime-reverse/libgles2-mx6

Without this patch they work fine building after other imx machines that do support gles2 before building imx6slevk.

I know Otavio needs this for mx53 but it needs more work for inclusion with machines that do not support gles2.

Lauren

-----Original Message-----
From: openembedded-core-bounces at lists.openembedded.org [mailto:openembedded-core-bounces at lists.openembedded.org] On Behalf Of Otavio Salvador
Sent: Sunday, October 05, 2014 10:14 AM
To: OpenEmbedded Core Mailing List
Cc: Otavio Salvador
Subject: [OE-core] [PATCH] package.bbclass: Reverse runtime symlinks should be tied to package generation

In case a package is not generated (is empty and does not has allow empty flag set) the package data regarding reverse runtime dependency shouldn't be done.

This were causing a false-positive in the meta-fsl-arm layer, when building mesa, as:

,----[ Error during build of MX53 in meta-fsl-arm ]
| ERROR: The recipe mesa is trying to install files into a shared area  
| when those files already exist. Those files and their manifest  
| location are:
|    /.../build/build/tmp/sysroots/imx53qsb/pkgdata/runtime-reverse/libopenvg-dev
|    Matched in manifest-imx53qsb-amd-gpu-x11-bin-mx51.packagedata
| Please verify which recipe should provide the above files.
`----

Fixes [YOCTO: #6795]

Signed-off-by: Otavio Salvador <otavio at ossystems.com.br>
---
 meta/classes/package.bbclass | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass index a877d28..f17c8d5 100644
--- a/meta/classes/package.bbclass
+++ b/meta/classes/package.bbclass
@@ -1241,10 +1241,6 @@ python emit_pkgdata() {
         sf.write('%s_%s: %d\n' % ('PKGSIZE', pkg, total_size))
         sf.close()
 
-        # Symlinks needed for reverse lookups (from the final package name)
-        subdata_sym = pkgdatadir + "/runtime-reverse/%s" % pkgval
-        oe.path.symlink("../runtime/%s" % pkg, subdata_sym, True)
-        
         # Symlinks needed for rprovides lookup
         if rprov:
             for p in rprov.strip().split():
@@ -1259,6 +1255,10 @@ python emit_pkgdata() {
         os.chdir(root)
         g = glob('*')
         if g or allow_empty == "1":
+            # Symlinks needed for reverse lookups (from the final package name)
+            subdata_sym = pkgdatadir + "/runtime-reverse/%s" % pkgval
+            oe.path.symlink("../runtime/%s" % pkg, subdata_sym, True)
+
             packagedfile = pkgdatadir + '/runtime/%s.packaged' % pkg
             open(packagedfile, 'w').close()
 
--
2.1.1

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