[oe-commits] Chris Larson : recipe_sanity: check RRECOMMENDS/ RCONFLICTS and make the message more comprehensible

[git version control]

Module: openembedded.git
Branch: org.openembedded.dev
Commit: a1ff5ef51501fa8f845cce44790f43bc77b89544
URL:    http://gitweb.openembedded.net/?p=openembedded.git&a=commit;h=a1ff5ef51501fa8f845cce44790f43bc77b89544

Author: Chris Larson <clarson at mvista.com>
Date:   Fri Jan 22 14:35:51 2010 -0700

recipe_sanity: check RRECOMMENDS/RCONFLICTS and make the message more comprehensible

Signed-off-by: Chris Larson <clarson at mvista.com>

---

 classes/recipe_sanity.bbclass |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/classes/recipe_sanity.bbclass b/classes/recipe_sanity.bbclass
index d929da6..9becf49 100644
--- a/classes/recipe_sanity.bbclass
+++ b/classes/recipe_sanity.bbclass
@@ -1,16 +1,16 @@
 def __note(msg, d):
     bb.note("%s: recipe_sanity: %s" % (d.getVar("P", 1), msg))
 
-__recipe_sanity_badtargetvars = "RDEPENDS RPROVIDES"
-def bad_target_vars(cfgdata, d):
+__recipe_sanity_badruntimevars = "RDEPENDS RPROVIDES RRECOMMENDS RCONFLICTS"
+def bad_runtime_vars(cfgdata, d):
     if bb.data.inherits_class("native", d) or \
        bb.data.inherits_class("cross", d):
         return
 
-    for var in d.getVar("__recipe_sanity_badtargetvars", 1).split():
+    for var in d.getVar("__recipe_sanity_badruntimevars", 1).split():
         val = d.getVar(var, 0)
         if val and val != cfgdata.get(var):
-            __note("%s should not be set, but is set to '%s'" % (var, val), d)
+            __note("%s should be %s_${PN}" % (var, var), d)
 
 __recipe_sanity_reqvars = "DESCRIPTION"
 __recipe_sanity_reqdiffvars = "LICENSE"
@@ -133,7 +133,7 @@ python do_recipe_sanity () {
     can_remove_others(p, cfgdata, d)
     var_renames_overwrite(cfgdata, d)
     req_vars(cfgdata, d)
-    bad_target_vars(cfgdata, d)
+    bad_runtime_vars(cfgdata, d)
 }
 do_recipe_sanity[nostamp] = "1"
 #do_recipe_sanity[recrdeptask] = "do_recipe_sanity"