]> code.ossystems Code Review - openembedded-core.git/commitdiff
features_check/insane: Use hasOverrides datastore method
authorRichard Purdie <richard.purdie@linuxfoundation.org>
Tue, 15 Feb 2022 23:09:43 +0000 (23:09 +0000)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 17 Feb 2022 17:48:10 +0000 (17:48 +0000)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/classes/features_check.bbclass
meta/classes/insane.bbclass

index 205e1b9cd3164d97c29dc8a53c85249dda1edaa1..3ef6b35baae8ec21ccd6be6ebbc3ded0b002ae61 100644 (file)
@@ -19,12 +19,9 @@ python () {
     unused = True
 
     for kind in ['DISTRO', 'MACHINE', 'COMBINED', 'IMAGE']:
-        if d.getVar('ANY_OF_' + kind + '_FEATURES') is None and \
-           d.overridedata.get('ANY_OF_' + kind + '_FEATURES') is None and \
-           d.getVar('REQUIRED_' + kind + '_FEATURES') is None and \
-           d.overridedata.get('REQUIRED_' + kind + '_FEATURES') is None and \
-           d.getVar('CONFLICT_' + kind + '_FEATURES') is None and \
-           d.overridedata.get('CONFLICT_' + kind + '_FEATURES') is None:
+        if d.getVar('ANY_OF_' + kind + '_FEATURES') is None and not d.hasOverrides('ANY_OF_' + kind + '_FEATURES') and \
+           d.getVar('REQUIRED_' + kind + '_FEATURES') is None and not d.hasOverrides('REQUIRED_' + kind + '_FEATURES') and \
+           d.getVar('CONFLICT_' + kind + '_FEATURES') is None and not d.hasOverrides('CONFLICT_' + kind + '_FEATURES'):
             continue
 
         unused = False
index a13a947bcf93595255204680ce36d8502c3cfb30..a28fcd479862d9e1e8ef457d06e10af5f1196dc5 100644 (file)
@@ -992,7 +992,7 @@ def package_qa_check_unhandled_features_check(pn, d, messages):
         var_set = False
         for kind in ['DISTRO', 'MACHINE', 'COMBINED']:
             for var in ['ANY_OF_' + kind + '_FEATURES', 'REQUIRED_' + kind + '_FEATURES', 'CONFLICT_' + kind + '_FEATURES']:
-                if d.getVar(var) is not None or d.overridedata.get(var) is not None:
+                if d.getVar(var) is not None or d.hasOverrides(var):
                     var_set = True
         if var_set:
             oe.qa.handle_error("unhandled-features-check", "%s: recipe doesn't inherit features_check" % pn, d)