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/testexport.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/testexport.bbclass')
-rw-r--r-- | meta/classes/testexport.bbclass | 40 |
1 files changed, 20 insertions, 20 deletions
diff --git a/meta/classes/testexport.bbclass b/meta/classes/testexport.bbclass index 5147020820..3f7b2de71d 100644 --- a/meta/classes/testexport.bbclass +++ b/meta/classes/testexport.bbclass | |||
@@ -49,19 +49,19 @@ def exportTests(d,tc): | |||
49 | import re | 49 | import re |
50 | import oe.path | 50 | import oe.path |
51 | 51 | ||
52 | exportpath = d.getVar("TEST_EXPORT_DIR", True) | 52 | exportpath = d.getVar("TEST_EXPORT_DIR") |
53 | 53 | ||
54 | savedata = {} | 54 | savedata = {} |
55 | savedata["d"] = {} | 55 | savedata["d"] = {} |
56 | savedata["target"] = {} | 56 | savedata["target"] = {} |
57 | savedata["target"]["ip"] = tc.target.ip or d.getVar("TEST_TARGET_IP", True) | 57 | savedata["target"]["ip"] = tc.target.ip or d.getVar("TEST_TARGET_IP") |
58 | savedata["target"]["server_ip"] = tc.target.server_ip or d.getVar("TEST_SERVER_IP", True) | 58 | savedata["target"]["server_ip"] = tc.target.server_ip or d.getVar("TEST_SERVER_IP") |
59 | 59 | ||
60 | keys = [ key for key in d.keys() if not key.startswith("_") and not key.startswith("BB") \ | 60 | keys = [ key for key in d.keys() if not key.startswith("_") and not key.startswith("BB") \ |
61 | and not key.startswith("B_pn") and not key.startswith("do_") and not d.getVarFlag(key, "func", True)] | 61 | and not key.startswith("B_pn") and not key.startswith("do_") and not d.getVarFlag(key, "func", True)] |
62 | for key in keys: | 62 | for key in keys: |
63 | try: | 63 | try: |
64 | savedata["d"][key] = d.getVar(key, True) | 64 | savedata["d"][key] = d.getVar(key) |
65 | except bb.data_smart.ExpansionError: | 65 | except bb.data_smart.ExpansionError: |
66 | # we don't care about those anyway | 66 | # we don't care about those anyway |
67 | pass | 67 | pass |
@@ -71,7 +71,7 @@ def exportTests(d,tc): | |||
71 | json.dump(savedata, f, skipkeys=True, indent=4, sort_keys=True) | 71 | json.dump(savedata, f, skipkeys=True, indent=4, sort_keys=True) |
72 | 72 | ||
73 | # Replace absolute path with relative in the file | 73 | # Replace absolute path with relative in the file |
74 | exclude_path = os.path.join(d.getVar("COREBASE", True),'meta','lib','oeqa') | 74 | exclude_path = os.path.join(d.getVar("COREBASE"),'meta','lib','oeqa') |
75 | f1 = open(json_file,'r').read() | 75 | f1 = open(json_file,'r').read() |
76 | f2 = open(json_file,'w') | 76 | f2 = open(json_file,'w') |
77 | m = f1.replace(exclude_path,'oeqa') | 77 | m = f1.replace(exclude_path,'oeqa') |
@@ -90,7 +90,7 @@ def exportTests(d,tc): | |||
90 | bb.utils.mkdirhier(os.path.join(exportpath, "oeqa/runtime/files")) | 90 | bb.utils.mkdirhier(os.path.join(exportpath, "oeqa/runtime/files")) |
91 | bb.utils.mkdirhier(os.path.join(exportpath, "oeqa/utils")) | 91 | bb.utils.mkdirhier(os.path.join(exportpath, "oeqa/utils")) |
92 | # copy test modules, this should cover tests in other layers too | 92 | # copy test modules, this should cover tests in other layers too |
93 | bbpath = d.getVar("BBPATH", True).split(':') | 93 | bbpath = d.getVar("BBPATH").split(':') |
94 | for t in tc.testslist: | 94 | for t in tc.testslist: |
95 | isfolder = False | 95 | isfolder = False |
96 | if re.search("\w+\.\w+\.test_\S+", t): | 96 | if re.search("\w+\.\w+\.test_\S+", t): |
@@ -111,7 +111,7 @@ def exportTests(d,tc): | |||
111 | if os.path.isfile(json_file): | 111 | if os.path.isfile(json_file): |
112 | shutil.copy2(json_file, os.path.join(exportpath, "oeqa/runtime")) | 112 | shutil.copy2(json_file, os.path.join(exportpath, "oeqa/runtime")) |
113 | # Get meta layer | 113 | # Get meta layer |
114 | for layer in d.getVar("BBLAYERS", True).split(): | 114 | for layer in d.getVar("BBLAYERS").split(): |
115 | if os.path.basename(layer) == "meta": | 115 | if os.path.basename(layer) == "meta": |
116 | meta_layer = layer | 116 | meta_layer = layer |
117 | break | 117 | break |
@@ -130,28 +130,28 @@ def exportTests(d,tc): | |||
130 | shutil.copy2(os.path.join(root, f), os.path.join(exportpath, "oeqa/runtime/files")) | 130 | shutil.copy2(os.path.join(root, f), os.path.join(exportpath, "oeqa/runtime/files")) |
131 | 131 | ||
132 | # Create tar file for common parts of testexport | 132 | # Create tar file for common parts of testexport |
133 | create_tarball(d, "testexport.tar.gz", d.getVar("TEST_EXPORT_DIR", True)) | 133 | create_tarball(d, "testexport.tar.gz", d.getVar("TEST_EXPORT_DIR")) |
134 | 134 | ||
135 | # Copy packages needed for runtime testing | 135 | # Copy packages needed for runtime testing |
136 | test_pkg_dir = d.getVar("TEST_NEEDED_PACKAGES_DIR", True) | 136 | test_pkg_dir = d.getVar("TEST_NEEDED_PACKAGES_DIR") |
137 | if os.listdir(test_pkg_dir): | 137 | if os.listdir(test_pkg_dir): |
138 | export_pkg_dir = os.path.join(d.getVar("TEST_EXPORT_DIR", True), "packages") | 138 | export_pkg_dir = os.path.join(d.getVar("TEST_EXPORT_DIR"), "packages") |
139 | oe.path.copytree(test_pkg_dir, export_pkg_dir) | 139 | oe.path.copytree(test_pkg_dir, export_pkg_dir) |
140 | # Create tar file for packages needed by the DUT | 140 | # Create tar file for packages needed by the DUT |
141 | create_tarball(d, "testexport_packages_%s.tar.gz" % d.getVar("MACHINE", True), export_pkg_dir) | 141 | create_tarball(d, "testexport_packages_%s.tar.gz" % d.getVar("MACHINE"), export_pkg_dir) |
142 | 142 | ||
143 | # Copy SDK | 143 | # Copy SDK |
144 | if d.getVar("TEST_EXPORT_SDK_ENABLED", True) == "1": | 144 | if d.getVar("TEST_EXPORT_SDK_ENABLED") == "1": |
145 | sdk_deploy = d.getVar("SDK_DEPLOY", True) | 145 | sdk_deploy = d.getVar("SDK_DEPLOY") |
146 | tarball_name = "%s.sh" % d.getVar("TEST_EXPORT_SDK_NAME", True) | 146 | tarball_name = "%s.sh" % d.getVar("TEST_EXPORT_SDK_NAME") |
147 | tarball_path = os.path.join(sdk_deploy, tarball_name) | 147 | tarball_path = os.path.join(sdk_deploy, tarball_name) |
148 | export_sdk_dir = os.path.join(d.getVar("TEST_EXPORT_DIR", True), | 148 | export_sdk_dir = os.path.join(d.getVar("TEST_EXPORT_DIR"), |
149 | d.getVar("TEST_EXPORT_SDK_DIR", True)) | 149 | d.getVar("TEST_EXPORT_SDK_DIR")) |
150 | bb.utils.mkdirhier(export_sdk_dir) | 150 | bb.utils.mkdirhier(export_sdk_dir) |
151 | shutil.copy2(tarball_path, export_sdk_dir) | 151 | shutil.copy2(tarball_path, export_sdk_dir) |
152 | 152 | ||
153 | # Create tar file for the sdk | 153 | # Create tar file for the sdk |
154 | create_tarball(d, "testexport_sdk_%s.tar.gz" % d.getVar("SDK_ARCH", True), export_sdk_dir) | 154 | create_tarball(d, "testexport_sdk_%s.tar.gz" % d.getVar("SDK_ARCH"), export_sdk_dir) |
155 | 155 | ||
156 | bb.plain("Exported tests to: %s" % exportpath) | 156 | bb.plain("Exported tests to: %s" % exportpath) |
157 | 157 | ||
@@ -161,8 +161,8 @@ def testexport_main(d): | |||
161 | from oeqa.utils.dump import get_host_dumper | 161 | from oeqa.utils.dump import get_host_dumper |
162 | 162 | ||
163 | test_create_extract_dirs(d) | 163 | test_create_extract_dirs(d) |
164 | export_dir = d.getVar("TEST_EXPORT_DIR", True) | 164 | export_dir = d.getVar("TEST_EXPORT_DIR") |
165 | bb.utils.mkdirhier(d.getVar("TEST_LOG_DIR", True)) | 165 | bb.utils.mkdirhier(d.getVar("TEST_LOG_DIR")) |
166 | bb.utils.remove(export_dir, recurse=True) | 166 | bb.utils.remove(export_dir, recurse=True) |
167 | bb.utils.mkdirhier(export_dir) | 167 | bb.utils.mkdirhier(export_dir) |
168 | 168 | ||
@@ -188,7 +188,7 @@ def create_tarball(d, tar_name, src_dir): | |||
188 | 188 | ||
189 | import tarfile | 189 | import tarfile |
190 | 190 | ||
191 | tar_path = os.path.join(d.getVar("TEST_EXPORT_DIR", True), tar_name) | 191 | tar_path = os.path.join(d.getVar("TEST_EXPORT_DIR"), tar_name) |
192 | current_dir = os.getcwd() | 192 | current_dir = os.getcwd() |
193 | src_dir = src_dir.rstrip('/') | 193 | src_dir = src_dir.rstrip('/') |
194 | dir_name = os.path.dirname(src_dir) | 194 | dir_name = os.path.dirname(src_dir) |