Skip to contentSkip to conversation

insane.bbclass: print more info for build-deps and file-rdeps

PATCH In review Robert Yang 1 participant 3 messages Jan 21, 2016 – Feb 18, 2016

lore
Share & tools
lore
#0 lore raw

cover insane.bbclass: print more info for build-deps and file-rdeps

The following changes since commit 8084c185c43cb7e7f5e29d31939dfd308f8a0f93:

udev: Add 2 patches to support 4.4 kernel (2016-01-20 09:20:06 +0000)

are available in the git repository at:

git://git.openembedded.org/openembedded-core-contrib rbt/insane
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=rbt/insane

Robert Yang (1):
insane.bbclass: print more info for build-deps and file-rdeps

 meta/classes/insane.bbclass |    9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)
Signature · 1 line
1.7.9.5
#1 lore raw

PATCH 1/1 insane.bbclass: print more info for build-deps and file-rdeps

This is useful for oe newbie:
* build-deps: print recipe name, and suggest fixing from DEPENDS or
PACKAGECONFIG, for example:
WARNING: QA Issue: patch rdepends on libattr, but it isn't a build dependency, missing attr in DEPENDS or PACKAGECONFIG? [build-deps]

* file-rdeps: print RDEPENDS_pkg rather than RDEPENDS, for example:
WARNING: QA Issue: /sbin/osd_login contained in package nfs-utils requires /bin/bash, but no providers found in RDEPENDS_nfs-utils? [file-rdeps]

Fix a bug in file-rdeps, there was "@underscore@" in file-rdeps:
WARNING: QA Issue: /sbin/osd@underscore@login_nfs-utils contained in [snip]

[YOCTO #8922]
[YOCTO #8847]

---
 meta/classes/insane.bbclass |    9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)
meta/classes/insane.bbclass +6 -3
index f7781ce..129e9dd 100644
@@ -821,7 +821,10 @@ def package_qa_check_rdepends(pkg, pkgdest, skip, taskdeps, packages, d):
                                 break
                     if rdep_data and 'PN' in rdep_data and rdep_data['PN'] in taskdeps:
                         continue
-                    error_msg = "%s rdepends on %s, but it isn't a build dependency?" % (pkg, rdepend)
+                    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'])
+                    else:
+                        error_msg = "%s rdepends on %s, but it isn't a build dependency?" % (pkg, rdepend)
                     package_qa_handle_error("build-deps", error_msg, d)
 
         if "file-rdeps" not in skip:
@@ -887,8 +890,8 @@ def package_qa_check_rdepends(pkg, pkgdest, skip, taskdeps, packages, d):
                         break
             if filerdepends:
                 for key in filerdepends:
-                    error_msg = "%s contained in package %s requires %s, but no providers found in its RDEPENDS" % \
-                            (filerdepends[key],pkg, key)
+                    error_msg = "%s contained in package %s requires %s, but no providers found in RDEPENDS_%s?" % \
+                            (filerdepends[key].replace("_%s" % pkg, "").replace("@underscore@", "_"), pkg, key, pkg)
                 package_qa_handle_error("file-rdeps", error_msg, d)
 
 def package_qa_check_deps(pkg, pkgdest, skip, d):
Signature · 1 line
1.7.9.5
#2 lore raw
In reply to Robert Yang

Ping.

// Robert

Quoted 15 lines · Robert Yang

The following changes since commit 8084c185c43cb7e7f5e29d31939dfd308f8a0f93:

   udev: Add 2 patches to support 4.4 kernel (2016-01-20 09:20:06 +0000)

are available in the git repository at:

git://git.openembedded.org/openembedded-core-contrib rbt/insane
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=rbt/insane

Robert Yang (1):
insane.bbclass: print more info for build-deps and file-rdeps

  meta/classes/insane.bbclass |    9 ++++++---
  1 file changed, 6 insertions(+), 3 deletions(-)