]> code.ossystems Code Review - openembedded-core.git/commitdiff
classes/package_rpm.bbclass: Change the way the PV is transformed
authorMark Hatle <mark.hatle@windriver.com>
Tue, 21 Jun 2011 00:08:19 +0000 (19:08 -0500)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 23 Jun 2011 14:59:39 +0000 (15:59 +0100)
There were some odd instances where the PKGV could not be loaded in the old
way.  Change to verify that PKGV exists before attempting to retrieve the
value from the key.

Signed-off-by: Mark Hatle <mark.hatle@windriver.com>
meta/classes/package_rpm.bbclass

index 9a854f383423c95d89c4d5be5b09b5e03ff94b27..2cc57424c75af25ff6c58cb488d802bbb77564b2 100644 (file)
@@ -318,8 +318,20 @@ python write_specfile () {
        import textwrap
        import oe.packagedata
 
-       # We need to change '-' in a version field to '+'
-       # This needs to be done BEFORE the mapping_rename_hook
+       # In RPM, dependencies are of the format: pkg <>= Epoch:Version-Release
+       # This format is similar to OE, however there are restrictions on the
+       # characters that can be in a field.  In the Version field, "-"
+       # characters are not allowed.  "-" is allowed in the Release field.
+       #
+       # We translate the "-" in the version to a "+", by loading the PKGV
+       # from the dependent recipe, replacing the - with a +, and then using
+       # that value to do a replace inside of this recipe's dependencies.
+       # This preserves the "-" separator between the version and release, as
+       # well as any "-" characters inside of the release field.
+       #
+       # All of this has to happen BEFORE the mapping_rename_hook as
+       # after renaming we cannot look up the dependencies in the packagedata
+       # store.
        def translate_vers(varname, d):
                depends = bb.data.getVar(varname, d, True)
                if depends:
@@ -330,9 +342,10 @@ python write_specfile () {
                                if dep and ver:
                                        if '-' in ver:
                                                subd = oe.packagedata.read_subpkgdata_dict(dep, d)
-                                               pv = subd['PKGV']
-                                               reppv = pv.replace('-', '+')
-                                               ver = ver.replace(pv, reppv)
+                                               if 'PKGV' in subd:
+                                                       pv = subd['PKGV']
+                                                       reppv = pv.replace('-', '+')
+                                                       ver = ver.replace(pv, reppv)
                                newdeps_dict[dep] = ver
                        depends = bb.utils.join_deps(newdeps_dict)
                        bb.data.setVar(varname, depends.strip(), d)