diff options
author | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2013-02-12 16:02:24 -0500 |
---|---|---|
committer | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2013-02-14 14:57:04 -0500 |
commit | 43984bc5ebc59bd8c5890ed6ba3de162e6698dcc (patch) | |
tree | d9e972b160d5d5033537495e9f507cfe043483c3 /src/lib/Bcfg2/Client/Tools/Pacman.py | |
parent | d526c8745b61c1977b775cfe750a86c51dd6a9f3 (diff) | |
download | bcfg2-43984bc5ebc59bd8c5890ed6ba3de162e6698dcc.tar.gz bcfg2-43984bc5ebc59bd8c5890ed6ba3de162e6698dcc.tar.bz2 bcfg2-43984bc5ebc59bd8c5890ed6ba3de162e6698dcc.zip |
better Executor class for client tools
Diffstat (limited to 'src/lib/Bcfg2/Client/Tools/Pacman.py')
-rw-r--r-- | src/lib/Bcfg2/Client/Tools/Pacman.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/lib/Bcfg2/Client/Tools/Pacman.py b/src/lib/Bcfg2/Client/Tools/Pacman.py index 9c14a3de6..12785afee 100644 --- a/src/lib/Bcfg2/Client/Tools/Pacman.py +++ b/src/lib/Bcfg2/Client/Tools/Pacman.py @@ -20,9 +20,8 @@ class Pacman(Bcfg2.Client.Tools.PkgTool): def RefreshPackages(self): '''Refresh memory hashes of packages''' - pkgcache = self.cmd.run("/usr/bin/pacman -Q")[1] self.installed = {} - for pkg in pkgcache: + for pkg in self.cmd.run("/usr/bin/pacman -Q").stdout.splitlines(): pkgname = pkg.split(' ')[0].strip() version = pkg.split(' ')[1].strip() #self.logger.info(" pkgname: %s, version: %s" % (pkgname, version)) @@ -62,7 +61,7 @@ class Pacman(Bcfg2.Client.Tools.PkgTool): '''Remove extra packages''' names = [pkg.get('name') for pkg in packages] self.logger.info("Removing packages: %s" % " ".join(names)) - self.cmd.run("%s --noconfirm --noprogressbar -R %s" % \ + self.cmd.run("%s --noconfirm --noprogressbar -R %s" % (self.pkgtool, " ".join(names))) self.RefreshPackages() self.extra = self.FindExtra() |