Skip to contentSkip to conversation

insane: check if the recipe incorrectly uses DEPENDS_${PN}

PATCH New Ross Burton 1 participant 1 message Aug 1, 2019

lore
Share & tools
lore

Ross Burton

author
#0 lore raw

PATCH 1/1 insane: check if the recipe incorrectly uses DEPENDS_${PN}

Some people mistakenly use DEPENDS_${PN} and wonder why the dependencies don't work. Check for this and tell the user to use DEPENDS.

Signed-off-by Ross Burton <[email protected]>
---
 meta/classes/insane.bbclass | 5 +++++
 1 file changed, 5 insertions(+)
meta/classes/insane.bbclass +5 -0
index 891d3a84218..35c4fdb4913 100644
@@ -1237,6 +1237,11 @@ python () {
     if prog.search(pn):
         package_qa_handle_error("uppercase-pn", 'PN: %s is upper case, this can result in unexpected behavior.' % pn, d)
 
+    # Some people mistakenly use DEPENDS_${PN} instead of DEPENDS and wonder
+    # why it doesn't work.
+    if (d.getVar(d.expand('DEPENDS_${PN}'))):
+        package_qa_handle_error("pkgvarcheck", "recipe uses DEPENDS_${PN}, should use DEPENDS", d)
+
     issues = []
     if (d.getVar('PACKAGES') or "").split():
         for dep in (d.getVar('QADEPENDS') or "").split():
Signature · 1 line
2.20.1