diff options
author | Ross Burton <ross.burton@intel.com> | 2017-09-13 10:58:09 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-09-13 21:59:57 +0100 |
commit | 0001ceead406b1e8ba4fd16d0ecb5fbf5b55ba66 (patch) | |
tree | 7018bf5a15768465b2b7d166d155e4d733406253 | |
parent | 340d413f678a4a64dfa060e8fe0ac721b73fed97 (diff) | |
download | openembedded-core-0001ceead406b1e8ba4fd16d0ecb5fbf5b55ba66.tar.gz openembedded-core-0001ceead406b1e8ba4fd16d0ecb5fbf5b55ba66.tar.bz2 openembedded-core-0001ceead406b1e8ba4fd16d0ecb5fbf5b55ba66.zip |
insane: don't pass skip list to functions which don't respect it
When these functions are being called INSANE_SKIP has already been taken into
account, so don't confuse the code by passing the skip list.
Signed-off-by: Ross Burton <ross.burton@intel.com>
-rw-r--r-- | meta/classes/insane.bbclass | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass index 78b41caf99..ebcfacc492 100644 --- a/meta/classes/insane.bbclass +++ b/meta/classes/insane.bbclass @@ -732,7 +732,7 @@ def package_qa_check_staged(path,d): return sane # Run all package-wide warnfuncs and errorfuncs -def package_qa_package(warnfuncs, errorfuncs, skip, package, d): +def package_qa_package(warnfuncs, errorfuncs, package, d): warnings = {} errors = {} @@ -749,7 +749,7 @@ def package_qa_package(warnfuncs, errorfuncs, skip, package, d): return len(errors) == 0 # Run all recipe-wide warnfuncs and errorfuncs -def package_qa_recipe(warnfuncs, errorfuncs, skip, pn, d): +def package_qa_recipe(warnfuncs, errorfuncs, pn, d): warnings = {} errors = {} @@ -766,7 +766,7 @@ def package_qa_recipe(warnfuncs, errorfuncs, skip, pn, d): return len(errors) == 0 # Walk over all files in a directory and call func -def package_qa_walk(warnfuncs, errorfuncs, skip, package, d): +def package_qa_walk(warnfuncs, errorfuncs, package, d): import oe.qa #if this will throw an exception, then fix the dict above @@ -908,7 +908,7 @@ def package_qa_check_rdepends(pkg, pkgdest, skip, taskdeps, packages, d): package_qa_handle_error("file-rdeps", error_msg, d) package_qa_check_rdepends[vardepsexclude] = "OVERRIDES" -def package_qa_check_deps(pkg, pkgdest, skip, d): +def package_qa_check_deps(pkg, pkgdest, d): localdata = bb.data.createCopy(d) localdata.setVar('OVERRIDES', pkg) @@ -1108,16 +1108,16 @@ python do_package_qa () { "%s doesn't match the [a-z0-9.+-]+ regex" % package, d) warn_checks, error_checks = parse_test_matrix("QAPATHTEST") - package_qa_walk(warn_checks, error_checks, skip, package, d) + package_qa_walk(warn_checks, error_checks, package, d) warn_checks, error_checks = parse_test_matrix("QAPKGTEST") - package_qa_package(warn_checks, error_checks, skip, package, d) + package_qa_package(warn_checks, error_checks, package, d) package_qa_check_rdepends(package, pkgdest, skip, taskdeps, packages, d) - package_qa_check_deps(package, pkgdest, skip, d) + package_qa_check_deps(package, pkgdest, d) warn_checks, error_checks = parse_test_matrix("QARECIPETEST") - package_qa_recipe(warn_checks, error_checks, skip, pn, d) + package_qa_recipe(warn_checks, error_checks, pn, d) if 'libdir' in d.getVar("ALL_QA").split(): package_qa_check_libdir(d) |