From 9b80c039add2bdd7ed70774a4c379030df6538a5 Mon Sep 17 00:00:00 2001 From: Joshua Watt Date: Tue, 10 Dec 2024 10:32:19 -0700 Subject: classes-global/insane: Handle case where RPROVIDER is also a provider The check to see if a provider of a given package is listed first checks for an exact match of the provider name. However, if this match existed, but didn't match in the task dependencies, it would not continue to look for other providers of package. This would manifest if one (non-virtual) recipe package RPROVIDES the name of a package produced by another recipe. Fix this, and also clean up the code to make it more readable by using a function to check if a runtime dependency is in the task dependencies. In addition, if no provider is found, list all the possible providers instead of the last one that was looked at. (From OE-Core rev: f13de6ab616eb1e38960a2296111febe2a9f4a28) Signed-off-by: Joshua Watt Signed-off-by: Richard Purdie --- meta/classes-global/insane.bbclass | 21 +++++++++++++-------- 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/meta/classes-global/insane.bbclass b/meta/classes-global/insane.bbclass index e54d2c1508..866cef6526 100644 --- a/meta/classes-global/insane.bbclass +++ b/meta/classes-global/insane.bbclass @@ -825,6 +825,12 @@ def package_qa_check_rdepends(pkg, pkgdest, skip, taskdeps, packages, d): # Now do the sanity check!!! if "build-deps" not in skip: + def check_rdep(rdep_data, possible_pn): + if rdep_data and "PN" in rdep_data: + possible_pn.add(rdep_data["PN"]) + return rdep_data["PN"] in taskdeps + return False + for rdepend in rdepends: if "-dbg" in rdepend and "debug-deps" not in skip: error_msg = "%s rdepends on %s" % (pkg,rdepend) @@ -833,17 +839,16 @@ def package_qa_check_rdepends(pkg, pkgdest, skip, taskdeps, packages, d): error_msg = "%s rdepends on %s" % (pkg, rdepend) oe.qa.handle_error("dev-deps", error_msg, d) if rdepend not in packages: + possible_pn = set() rdep_data = oe.packagedata.read_subpkgdata(rdepend, d) - if rdep_data and 'PN' in rdep_data and rdep_data['PN'] in taskdeps: + if check_rdep(rdep_data, possible_pn): continue - if not rdep_data or not 'PN' in rdep_data: - for _, rdep_data in oe.packagedata.foreach_runtime_provider_pkgdata(d, rdepend): - if rdep_data and 'PN' in rdep_data and rdep_data['PN'] in taskdeps: - break - if rdep_data and 'PN' in rdep_data and rdep_data['PN'] in taskdeps: + + if any(check_rdep(rdep_data, possible_pn) for _, rdep_data in oe.packagedata.foreach_runtime_provider_pkgdata(d, rdepend)): continue - if rdep_data and 'PN' in rdep_data: - error_msg = "%s rdepends on %s, but it isn't a build dependency, missing %s in DEPENDS or PACKAGECONFIG?" % (pkg, rdepend, rdep_data['PN']) + + if possible_pn: + error_msg = "%s rdepends on %s, but it isn't a build dependency, missing one of %s in DEPENDS or PACKAGECONFIG?" % (pkg, rdepend, ", ".join(possible_pn)) else: error_msg = "%s rdepends on %s, but it isn't a build dependency?" % (pkg, rdepend) oe.qa.handle_error("build-deps", error_msg, d) -- cgit v1.2.3-54-g00ecf