diff options
| author | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2013-02-21 08:47:59 -0500 |
|---|---|---|
| committer | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2013-02-21 08:47:59 -0500 |
| commit | acb1dde9ba48b04d1ceb701ce849e96cef3d0070 (patch) | |
| tree | 4754d07af191cc08dfc2be55420403fdb887ad7a /src/lib/Bcfg2/Client/Tools/APT.py | |
| parent | d8002c666c6a450e99c9fe476a5a3dcfb23f05db (diff) | |
| download | bcfg2-acb1dde9ba48b04d1ceb701ce849e96cef3d0070.tar.gz bcfg2-acb1dde9ba48b04d1ceb701ce849e96cef3d0070.tar.bz2 bcfg2-acb1dde9ba48b04d1ceb701ce849e96cef3d0070.zip | |
removed in-place modification of "states" dict in client tools
Diffstat (limited to 'src/lib/Bcfg2/Client/Tools/APT.py')
| -rw-r--r-- | src/lib/Bcfg2/Client/Tools/APT.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/lib/Bcfg2/Client/Tools/APT.py b/src/lib/Bcfg2/Client/Tools/APT.py index e44668bf2..cc2f657d0 100644 --- a/src/lib/Bcfg2/Client/Tools/APT.py +++ b/src/lib/Bcfg2/Client/Tools/APT.py @@ -217,7 +217,7 @@ class APT(Bcfg2.Client.Tools.Tool): self.modified += packages self.extra = self.FindExtra() - def Install(self, packages, states): + def Install(self, packages): # it looks like you can't install arbitrary versions of software # out of the pkg cache, we will still need to call apt-get ipkgs = [] @@ -257,10 +257,12 @@ class APT(Bcfg2.Client.Tools.Tool): self.logger.error("APT command failed") self.pkg_cache = apt.cache.Cache() self.extra = self.FindExtra() + states = dict() for package in packages: states[package] = self.VerifyPackage(package, [], checksums=False) if states[package]: self.modified.append(package) + return states def VerifyPath(self, entry, _): """Do nothing here since we only verify Path type=ignore.""" |