[meta-virtualization] [[m-c-s][PATCH] housekeeping: drop usage of bb.data.getVar()

Mark Asselstine mark.asselstine at windriver.com
Mon Dec 19 12:10:37 PST 2016


Similar to oe-core commit 2864ff6a4b3c3f9b3bbb6d2597243cc5d3715939 the
bb.data.getVar() have been deprecated for enough time now that it has
been removed. We need to switch to the new getVar() to get things
working again.

Signed-off-by: Mark Asselstine <mark.asselstine at windriver.com>
---
 meta-openstack/classes/monitor.bbclass            | 22 +++++++++---------
 meta-openstack/classes/openstackchef_inc.bbclass  | 28 +++++++++++------------
 meta-openstack/recipes-devtools/erlang/erlang.inc |  2 +-
 3 files changed, 26 insertions(+), 26 deletions(-)

diff --git a/meta-openstack/classes/monitor.bbclass b/meta-openstack/classes/monitor.bbclass
index a959f09..e916137 100644
--- a/meta-openstack/classes/monitor.bbclass
+++ b/meta-openstack/classes/monitor.bbclass
@@ -12,7 +12,7 @@ addtask monitor_clean before do_clean
 def copy_check_files(d, check_var_name, src, dest):
     import shutil
 
-    mon_checks = bb.data.getVar(check_var_name, d, 1)
+    mon_checks = d.getVar(check_var_name, 1)
     if mon_checks != None:
         if not os.path.exists(dest):
             os.makedirs(dest)
@@ -29,9 +29,9 @@ python do_monitor_install() {
         bb.debug(1, 'OpenStack monitoring feature is disabled, skipping do_monitor_install')
         return
 
-    mon_dir = bb.data.getVar('MONITOR_STAGING_DIR', d, 1)
-    mon_services_dir = bb.data.getVar('MONITOR_STAGING_SERVICES_DIR', d, 1)
-    mon_checks_dir = bb.data.getVar('MONITOR_STAGING_CHECKS_DIR', d, 1)
+    mon_dir = d.getVar('MONITOR_STAGING_DIR', 1)
+    mon_services_dir = d.getVar('MONITOR_STAGING_SERVICES_DIR', 1)
+    mon_checks_dir = d.getVar('MONITOR_STAGING_CHECKS_DIR', 1)
     if not os.path.exists(mon_dir):
         os.makedirs(mon_dir)
     if not os.path.exists(mon_services_dir):
@@ -41,13 +41,13 @@ python do_monitor_install() {
     workdir = d.getVar('WORKDIR', True)
 
     # Process monitor SERVICE catagory
-    mon_service_pkgs = bb.data.getVar('MONITOR_SERVICE_PACKAGES', d, 1)
+    mon_service_pkgs = d.getVar('MONITOR_SERVICE_PACKAGES', 1)
     if mon_service_pkgs != None:
         for pkg in mon_service_pkgs.split():
             f_name = os.path.join(mon_services_dir, pkg + '.service')
             if os.path.exists(f_name):
                 os.remove(f_name)
-            data = bb.data.getVar('MONITOR_SERVICE_' + pkg, d, 1)
+            data = d.getVar('MONITOR_SERVICE_' + pkg, 1)
             if data != None:
                 f = open(f_name, 'w')
                 f.write(d.getVar('MONITOR_SERVICE_' + pkg))
@@ -62,14 +62,14 @@ python do_monitor_install() {
 python do_monitor_clean() {
     import shutil
 
-    mon_dir = bb.data.getVar('MONITOR_STAGING_DIR', d, 1)
-    mon_services_dir = bb.data.getVar('MONITOR_STAGING_SERVICES_DIR', d, 1)
-    mon_checks_dir = bb.data.getVar('MONITOR_STAGING_CHECKS_DIR', d, 1)
+    mon_dir = d.getVar('MONITOR_STAGING_DIR', 1)
+    mon_services_dir = d.getVar('MONITOR_STAGING_SERVICES_DIR', 1)
+    mon_checks_dir = d.getVar('MONITOR_STAGING_CHECKS_DIR', 1)
     if not os.path.exists(mon_dir):
         return
 
     # Clean up monitor SERVICE catagory
-    mon_service_pkgs = bb.data.getVar('MONITOR_SERVICE_PACKAGES', d, 1)
+    mon_service_pkgs = d.getVar('MONITOR_SERVICE_PACKAGES', 1)
     if mon_service_pkgs != None and os.path.exists(mon_services_dir):
         for pkg in mon_service_pkgs.split():
             f_name = os.path.join(mon_services_dir, pkg + '.service')
@@ -80,7 +80,7 @@ python do_monitor_clean() {
     packages = (d.getVar('PACKAGES', True) or "").split()
     if len(packages) >= 1 and os.path.exists(mon_checks_dir):
         for pkg in packages:
-            if bb.data.getVar('MONITOR_CHECKS_' + pkg, d, 1) != None:
+            if d.getVar('MONITOR_CHECKS_' + pkg, 1) != None:
                 shutil.rmtree(mon_checks_dir + "/" + pkg, True)
 }
 
diff --git a/meta-openstack/classes/openstackchef_inc.bbclass b/meta-openstack/classes/openstackchef_inc.bbclass
index 048fce6..91adb41 100644
--- a/meta-openstack/classes/openstackchef_inc.bbclass
+++ b/meta-openstack/classes/openstackchef_inc.bbclass
@@ -184,12 +184,12 @@ def deploychef_make_startup_shutdown_list(d):
 
     import os
     if d.getVar('INITSCRIPT_PACKAGES') or d.getVar('INITSCRIPT_NAME'):
-        #script_list = bb.data.getVar('INITSCRIPT_PACKAGES', d , 1)
+        #script_list = d.getVar('INITSCRIPT_PACKAGES', 1)
         script_list = d.getVar('INITSCRIPT_PACKAGES', True) or \ 
          d.getVar('INITSCRIPT_NAME', True)  #Some package do not define INITSCRIPT_PACKAGES
         msg="list of start/stop services: %s" % str(script_list)
         bb.debug(2, msg)
-        base_dir = bb.data.getVar('CHEF_TEMPLATE_BASE', d, 1 )
+        base_dir = d.getVar('CHEF_TEMPLATE_BASE', 1 )
         if not os.path.exists(base_dir):
             os.mkdir(base_dir)
         startup_file = os.path.join(base_dir, d.getVar('SRCNAME', True) +'-startup-list')
@@ -311,15 +311,15 @@ def deploychef_make_substitutions(d, sub_dict, attr_filename, sed_filename):
             hFile.close()
             if False:
                 msg = "Cannot read/write to attributes file %s as expected:%s"\
-                % (attr_filename, bb.data.getVar('FILE', d))
+                % (attr_filename, d.getVar('FILE'))
                 bb.build.FuncFailed(msg)
         else:
             msg = "The substitution dictionary variable sub_dict is not set  %s as expected "\
-            % bb.data.getVar('FILE')
+            % d.getVar('FILE')
             bb.error(msg)
     else:
         msg = "Null file names passsed to function %s %s "\
-        % (bb.data.getVar('FUNC',d), bb.data.getVar('FILE', d))
+        % (d.getVar('FUNC'), d.getVar('FILE'))
         bb.error(msg)
 
 
@@ -338,7 +338,7 @@ def deploychef_make_templates( d, conf_tuple=tuple()):
 
     if len(conf_tuple):
         import os, ast
-        base_dir = bb.data.getVar('CHEF_TEMPLATE_BASE', d, 1 )
+        base_dir = d.getVar('CHEF_TEMPLATE_BASE', 1 )
         attr_file = os.path.join(base_dir, d.getVar('SRCNAME', True) + '-attributes.rb')
         msg ="Default attributes saved to %s" % attr_file
         if os.path.exists(attr_file):
@@ -357,9 +357,9 @@ def deploychef_make_templates( d, conf_tuple=tuple()):
                 
                 #Make the necessary susbstitutions using the default
                 #substitutiin dictionary
-                sub_dict = bb.data.getVar('CHEF_SERVICES_DEFAULT_CONF_SUBS', d , 1)
+                sub_dict = d.getVar('CHEF_SERVICES_DEFAULT_CONF_SUBS', 1)
                 msg = "The variable %s is not set in %s as a dictionary as expected "\
-                % ('CHEF_SERVICES_DEFAULT_CONF_SUBS', bb.data.getVar('FILE', d))
+                % ('CHEF_SERVICES_DEFAULT_CONF_SUBS', d.getVar('FILE'))
                 if sub_dict:
                     #Safely retrieve our python data structure
                     sub_dict = ast.literal_eval(sub_dict)
@@ -371,11 +371,11 @@ def deploychef_make_templates( d, conf_tuple=tuple()):
                     raise bb.build.FuncFailed(msg)
                 #Make the necessary susbstitutions using auxilliary dictionary
                 #if provided by inheriting class
-                sub_dict = bb.data.getVar('CHEF_SERVICES_CONF_SUBS', d , 1)
+                sub_dict = d.getVar('CHEF_SERVICES_CONF_SUBS', 1)
                 if sub_dict:
                     sub_dict = ast.literal_eval(sub_dict)
                     msg = "The variable %s is not set in %s as a dictionary as expected "\
-                    % ('CHEF_SERVICES_CONF_SUB', bb.data.getVar('FILE', d))
+                    % ('CHEF_SERVICES_CONF_SUB', d.getVar('FILE'))
                     if type(sub_dict) is dict:
                         pass
                         deploychef_make_substitutions(d, sub_dict, attr_file, file_name)
@@ -530,11 +530,11 @@ def deploychef_postinst_substitutions(d, sub_dict, postinst):
                 hFile.close()
         else:
             msg = "The substitution dictionary variable sub_dict is not set  %s as expected "\
-            % bb.data.getVar('FILE')
+            % d.getVar('FILE')
             bb.build.FuncFailed(msg)
     else:
         msg = "Null string passsed to function %s %s "\
-        % (bb.data.getVar('FUNC',d), bb.data.getVar('FILE', d))
+        % (d.getVar('FUNC'), d.getVar('FILE'))
         bb.build.FuncFailed(msg)
     return postinst
 
@@ -565,9 +565,9 @@ def deploychef_update_package_postinsts(d):
             overrides = d.getVar("OVERRIDES", True)
             msg = "%s The override variable is %s :\n %s " % (pkg, overrides, postinst)
             bb.note(msg)
-            sub_dict = bb.data.getVar('CHEF_SERVICES_DEFAULT_CONF_SUBS', d , 1)
+            sub_dict = d.getVar('CHEF_SERVICES_DEFAULT_CONF_SUBS', 1)
             msg = "The variable %s is not set in %s as a dictionary as expected "\
-            % ('CHEF_SERVICES_DEFAULT_CONF_SUBS', bb.data.getVar('FILE', d))
+            % ('CHEF_SERVICES_DEFAULT_CONF_SUBS', d.getVar('FILE'))
             if sub_dict:
                 import ast
                 #Safely retrieve our python data structure
diff --git a/meta-openstack/recipes-devtools/erlang/erlang.inc b/meta-openstack/recipes-devtools/erlang/erlang.inc
index 8a7bd5a..37446d7 100644
--- a/meta-openstack/recipes-devtools/erlang/erlang.inc
+++ b/meta-openstack/recipes-devtools/erlang/erlang.inc
@@ -6,7 +6,7 @@ PRIORITY = "optional"
 LICENSE = "ErlPL-1.1"
 LIC_FILES_CHKSUM = "file://EPLICENCE;md5=09f9063ea35bc5bd124df2fda1d9d2c7"
 
-UPSTREAM_VERSION = "${@bb.data.getVar('PV', d, 1).replace('.', '-')}"
+UPSTREAM_VERSION = "${@d.getVar('PV', 1).replace('.', '-')}"
 
 SRC_URI = "http://erlang.org/download/otp_src_${UPSTREAM_VERSION}.tar.gz \
     file://erlang-fix-build-issue-in-Yocto.patch \
-- 
2.7.4



More information about the meta-virtualization mailing list