diff options
author | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2012-08-03 15:38:17 -0400 |
---|---|---|
committer | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2012-08-03 15:38:27 -0400 |
commit | 913d320ee02d5e16b61a14d97b2b1111d307d0d3 (patch) | |
tree | 64411709ef9a914016ad2f6c1f8146b92e2a5d3b /src/lib/Bcfg2/Client/Tools/RPMng.py | |
parent | fc296f452029120c79c25444195f346855ed6dde (diff) | |
download | bcfg2-913d320ee02d5e16b61a14d97b2b1111d307d0d3.tar.gz bcfg2-913d320ee02d5e16b61a14d97b2b1111d307d0d3.tar.bz2 bcfg2-913d320ee02d5e16b61a14d97b2b1111d307d0d3.zip |
removed Bcfg2.Client.XML abstraction layer since we depend directly on lxml.etree
Diffstat (limited to 'src/lib/Bcfg2/Client/Tools/RPMng.py')
-rw-r--r-- | src/lib/Bcfg2/Client/Tools/RPMng.py | 27 |
1 files changed, 14 insertions, 13 deletions
diff --git a/src/lib/Bcfg2/Client/Tools/RPMng.py b/src/lib/Bcfg2/Client/Tools/RPMng.py index 91e2180ae..a7fbc0036 100644 --- a/src/lib/Bcfg2/Client/Tools/RPMng.py +++ b/src/lib/Bcfg2/Client/Tools/RPMng.py @@ -1,8 +1,9 @@ """Bcfg2 Support for RPMS""" -import os.path +import os import rpm import rpmtools +import lxml.etree import Bcfg2.Client.Tools class RPMng(Bcfg2.Client.Tools.PkgTool): @@ -148,7 +149,7 @@ class RPMng(Bcfg2.Client.Tools.PkgTool): instances = [inst for inst in entry if inst.tag == 'Instance' or inst.tag == 'Package'] if instances == []: # We have an old style no Instance entry. Convert it to new style. - instance = Bcfg2.Client.XML.SubElement(entry, 'Package') + instance = lxml.etree.SubElement(entry, 'Package') for attrib in list(entry.attrib.keys()): instance.attrib[attrib] = entry.attrib[attrib] if (self.pkg_checks and @@ -849,14 +850,14 @@ class RPMng(Bcfg2.Client.Tools.PkgTool): for (name, instances) in list(self.installed.items()): if name not in packages: - extra_entry = Bcfg2.Client.XML.Element('Package', name=name, type=self.pkgtype) + extra_entry = lxml.etree.Element('Package', name=name, type=self.pkgtype) for installed_inst in instances: if self.setup['extra']: self.logger.info("Extra Package %s %s." % \ (name, self.str_evra(installed_inst))) - tmp_entry = Bcfg2.Client.XML.SubElement(extra_entry, 'Instance', \ - version = installed_inst.get('version'), \ - release = installed_inst.get('release')) + tmp_entry = lxml.etree.SubElement(extra_entry, 'Instance', + version = installed_inst.get('version'), + release = installed_inst.get('release')) if installed_inst.get('epoch', None) != None: tmp_entry.set('epoch', str(installed_inst.get('epoch'))) if installed_inst.get('arch', None) != None: @@ -873,7 +874,7 @@ class RPMng(Bcfg2.Client.Tools.PkgTool): """ name = pkg_entry.get('name') - extra_entry = Bcfg2.Client.XML.Element('Package', name=name, type=self.pkgtype) + extra_entry = lxml.etree.Element('Package', name=name, type=self.pkgtype) instances = [inst for inst in pkg_entry if inst.tag == 'Instance' or inst.tag == 'Package'] if name in self.installOnlyPkgs: for installed_inst in installed_entry: @@ -887,9 +888,9 @@ class RPMng(Bcfg2.Client.Tools.PkgTool): # Extra package. self.logger.info("Extra InstallOnlyPackage %s %s." % \ (name, self.str_evra(installed_inst))) - tmp_entry = Bcfg2.Client.XML.SubElement(extra_entry, 'Instance', \ - version = installed_inst.get('version'), \ - release = installed_inst.get('release')) + tmp_entry = lxml.etree.SubElement(extra_entry, 'Instance', + version = installed_inst.get('version'), + release = installed_inst.get('release')) if installed_inst.get('epoch', None) != None: tmp_entry.set('epoch', str(installed_inst.get('epoch'))) if installed_inst.get('arch', None) != None: @@ -906,9 +907,9 @@ class RPMng(Bcfg2.Client.Tools.PkgTool): if not_found: self.logger.info("Extra Normal Package Instance %s %s" % \ (name, self.str_evra(installed_inst))) - tmp_entry = Bcfg2.Client.XML.SubElement(extra_entry, 'Instance', \ - version = installed_inst.get('version'), \ - release = installed_inst.get('release')) + tmp_entry = lxml.etree.SubElement(extra_entry, 'Instance', + version = installed_inst.get('version'), + release = installed_inst.get('release')) if installed_inst.get('epoch', None) != None: tmp_entry.set('epoch', str(installed_inst.get('epoch'))) if installed_inst.get('arch', None) != None: |