diff options
author | Joshua Lock <joshua.g.lock@intel.com> | 2016-12-14 21:13:04 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-12-16 10:23:23 +0000 |
commit | c4e2c59088765d1f1de7ec57cde91980f887c2ff (patch) | |
tree | a2fda8ac5916fb59a711e9220c2177008cca9347 /meta/classes/multilib.bbclass | |
parent | d5e67725ac11e3296cad104470931ffa16824b90 (diff) | |
download | poky-c4e2c59088765d1f1de7ec57cde91980f887c2ff.tar.gz |
meta: remove True option to getVar calls
getVar() now defaults to expanding by default, thus remove the True
option from getVar() calls with a regex search and replace.
Search made with the following regex: getVar ?\(( ?[^,()]*), True\)
(From OE-Core rev: 7c552996597faaee2fbee185b250c0ee30ea3b5f)
Signed-off-by: Joshua Lock <joshua.g.lock@intel.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/classes/multilib.bbclass')
-rw-r--r-- | meta/classes/multilib.bbclass | 32 |
1 files changed, 16 insertions, 16 deletions
diff --git a/meta/classes/multilib.bbclass b/meta/classes/multilib.bbclass index d5a31287a8..498f7f5735 100644 --- a/meta/classes/multilib.bbclass +++ b/meta/classes/multilib.bbclass | |||
@@ -1,20 +1,20 @@ | |||
1 | python multilib_virtclass_handler () { | 1 | python multilib_virtclass_handler () { |
2 | cls = e.data.getVar("BBEXTENDCURR", True) | 2 | cls = e.data.getVar("BBEXTENDCURR") |
3 | variant = e.data.getVar("BBEXTENDVARIANT", True) | 3 | variant = e.data.getVar("BBEXTENDVARIANT") |
4 | if cls != "multilib" or not variant: | 4 | if cls != "multilib" or not variant: |
5 | return | 5 | return |
6 | 6 | ||
7 | e.data.setVar('STAGING_KERNEL_DIR', e.data.getVar('STAGING_KERNEL_DIR', True)) | 7 | e.data.setVar('STAGING_KERNEL_DIR', e.data.getVar('STAGING_KERNEL_DIR')) |
8 | 8 | ||
9 | # There should only be one kernel in multilib configs | 9 | # There should only be one kernel in multilib configs |
10 | # We also skip multilib setup for module packages. | 10 | # We also skip multilib setup for module packages. |
11 | provides = (e.data.getVar("PROVIDES", True) or "").split() | 11 | provides = (e.data.getVar("PROVIDES") or "").split() |
12 | if "virtual/kernel" in provides or bb.data.inherits_class('module-base', e.data): | 12 | if "virtual/kernel" in provides or bb.data.inherits_class('module-base', e.data): |
13 | raise bb.parse.SkipPackage("We shouldn't have multilib variants for the kernel") | 13 | raise bb.parse.SkipPackage("We shouldn't have multilib variants for the kernel") |
14 | 14 | ||
15 | save_var_name=e.data.getVar("MULTILIB_SAVE_VARNAME", True) or "" | 15 | save_var_name=e.data.getVar("MULTILIB_SAVE_VARNAME") or "" |
16 | for name in save_var_name.split(): | 16 | for name in save_var_name.split(): |
17 | val=e.data.getVar(name, True) | 17 | val=e.data.getVar(name) |
18 | if val: | 18 | if val: |
19 | e.data.setVar(name + "_MULTILIB_ORIGINAL", val) | 19 | e.data.setVar(name + "_MULTILIB_ORIGINAL", val) |
20 | 20 | ||
@@ -26,7 +26,7 @@ python multilib_virtclass_handler () { | |||
26 | if bb.data.inherits_class('image', e.data): | 26 | if bb.data.inherits_class('image', e.data): |
27 | e.data.setVar("MLPREFIX", variant + "-") | 27 | e.data.setVar("MLPREFIX", variant + "-") |
28 | e.data.setVar("PN", variant + "-" + e.data.getVar("PN", False)) | 28 | e.data.setVar("PN", variant + "-" + e.data.getVar("PN", False)) |
29 | e.data.setVar('SDKTARGETSYSROOT', e.data.getVar('SDKTARGETSYSROOT', True)) | 29 | e.data.setVar('SDKTARGETSYSROOT', e.data.getVar('SDKTARGETSYSROOT')) |
30 | target_vendor = e.data.getVar("TARGET_VENDOR_" + "virtclass-multilib-" + variant, False) | 30 | target_vendor = e.data.getVar("TARGET_VENDOR_" + "virtclass-multilib-" + variant, False) |
31 | if target_vendor: | 31 | if target_vendor: |
32 | e.data.setVar("TARGET_VENDOR", target_vendor) | 32 | e.data.setVar("TARGET_VENDOR", target_vendor) |
@@ -50,7 +50,7 @@ python multilib_virtclass_handler () { | |||
50 | 50 | ||
51 | 51 | ||
52 | # Expand this since this won't work correctly once we set a multilib into place | 52 | # Expand this since this won't work correctly once we set a multilib into place |
53 | e.data.setVar("ALL_MULTILIB_PACKAGE_ARCHS", e.data.getVar("ALL_MULTILIB_PACKAGE_ARCHS", True)) | 53 | e.data.setVar("ALL_MULTILIB_PACKAGE_ARCHS", e.data.getVar("ALL_MULTILIB_PACKAGE_ARCHS")) |
54 | 54 | ||
55 | override = ":virtclass-multilib-" + variant | 55 | override = ":virtclass-multilib-" + variant |
56 | 56 | ||
@@ -60,7 +60,7 @@ python multilib_virtclass_handler () { | |||
60 | 60 | ||
61 | # Expand the WHITELISTs with multilib prefix | 61 | # Expand the WHITELISTs with multilib prefix |
62 | for whitelist in ["WHITELIST_GPL-3.0", "LGPLv2_WHITELIST_GPL-3.0"]: | 62 | for whitelist in ["WHITELIST_GPL-3.0", "LGPLv2_WHITELIST_GPL-3.0"]: |
63 | pkgs = e.data.getVar(whitelist, True) | 63 | pkgs = e.data.getVar(whitelist) |
64 | for pkg in pkgs.split(): | 64 | for pkg in pkgs.split(): |
65 | pkgs += " " + variant + "-" + pkg | 65 | pkgs += " " + variant + "-" + pkg |
66 | e.data.setVar(whitelist, pkgs) | 66 | e.data.setVar(whitelist, pkgs) |
@@ -78,7 +78,7 @@ multilib_virtclass_handler[eventmask] = "bb.event.RecipePreFinalise" | |||
78 | STAGINGCC_prepend = "${BBEXTENDVARIANT}-" | 78 | STAGINGCC_prepend = "${BBEXTENDVARIANT}-" |
79 | 79 | ||
80 | python __anonymous () { | 80 | python __anonymous () { |
81 | variant = d.getVar("BBEXTENDVARIANT", True) | 81 | variant = d.getVar("BBEXTENDVARIANT") |
82 | 82 | ||
83 | import oe.classextend | 83 | import oe.classextend |
84 | 84 | ||
@@ -88,7 +88,7 @@ python __anonymous () { | |||
88 | clsextend.map_depends_variable("PACKAGE_INSTALL") | 88 | clsextend.map_depends_variable("PACKAGE_INSTALL") |
89 | clsextend.map_depends_variable("LINGUAS_INSTALL") | 89 | clsextend.map_depends_variable("LINGUAS_INSTALL") |
90 | clsextend.map_depends_variable("RDEPENDS") | 90 | clsextend.map_depends_variable("RDEPENDS") |
91 | pinstall = d.getVar("LINGUAS_INSTALL", True) + " " + d.getVar("PACKAGE_INSTALL", True) | 91 | pinstall = d.getVar("LINGUAS_INSTALL") + " " + d.getVar("PACKAGE_INSTALL") |
92 | d.setVar("PACKAGE_INSTALL", pinstall) | 92 | d.setVar("PACKAGE_INSTALL", pinstall) |
93 | d.setVar("LINGUAS_INSTALL", "") | 93 | d.setVar("LINGUAS_INSTALL", "") |
94 | # FIXME, we need to map this to something, not delete it! | 94 | # FIXME, we need to map this to something, not delete it! |
@@ -104,7 +104,7 @@ python __anonymous () { | |||
104 | return | 104 | return |
105 | 105 | ||
106 | clsextend.rename_packages() | 106 | clsextend.rename_packages() |
107 | clsextend.rename_package_variables((d.getVar("PACKAGEVARS", True) or "").split()) | 107 | clsextend.rename_package_variables((d.getVar("PACKAGEVARS") or "").split()) |
108 | 108 | ||
109 | clsextend.map_packagevars() | 109 | clsextend.map_packagevars() |
110 | clsextend.map_regexp_variable("PACKAGES_DYNAMIC") | 110 | clsextend.map_regexp_variable("PACKAGES_DYNAMIC") |
@@ -119,7 +119,7 @@ PACKAGEFUNCS_append = " do_package_qa_multilib" | |||
119 | python do_package_qa_multilib() { | 119 | python do_package_qa_multilib() { |
120 | 120 | ||
121 | def check_mlprefix(pkg, var, mlprefix): | 121 | def check_mlprefix(pkg, var, mlprefix): |
122 | values = bb.utils.explode_deps(d.getVar('%s_%s' % (var, pkg), True) or d.getVar(var, True) or "") | 122 | values = bb.utils.explode_deps(d.getVar('%s_%s' % (var, pkg), True) or d.getVar(var) or "") |
123 | candidates = [] | 123 | candidates = [] |
124 | for i in values: | 124 | for i in values: |
125 | if i.startswith('virtual/'): | 125 | if i.startswith('virtual/'): |
@@ -130,14 +130,14 @@ python do_package_qa_multilib() { | |||
130 | candidates.append(i) | 130 | candidates.append(i) |
131 | if len(candidates) > 0: | 131 | if len(candidates) > 0: |
132 | msg = "%s package %s - suspicious values '%s' in %s" \ | 132 | msg = "%s package %s - suspicious values '%s' in %s" \ |
133 | % (d.getVar('PN', True), pkg, ' '.join(candidates), var) | 133 | % (d.getVar('PN'), pkg, ' '.join(candidates), var) |
134 | package_qa_handle_error("multilib", msg, d) | 134 | package_qa_handle_error("multilib", msg, d) |
135 | 135 | ||
136 | ml = d.getVar('MLPREFIX', True) | 136 | ml = d.getVar('MLPREFIX') |
137 | if not ml: | 137 | if not ml: |
138 | return | 138 | return |
139 | 139 | ||
140 | packages = d.getVar('PACKAGES', True) | 140 | packages = d.getVar('PACKAGES') |
141 | for pkg in packages.split(): | 141 | for pkg in packages.split(): |
142 | check_mlprefix(pkg, 'RDEPENDS', ml) | 142 | check_mlprefix(pkg, 'RDEPENDS', ml) |
143 | check_mlprefix(pkg, 'RPROVIDES', ml) | 143 | check_mlprefix(pkg, 'RPROVIDES', ml) |