diff options
| -rw-r--r-- | src/lib/Client/Frame.py | 12 | ||||
| -rw-r--r-- | src/lib/Client/Tools/APT.py | 3 | ||||
| -rw-r--r-- | src/lib/Client/Tools/RPM.py | 3 |
3 files changed, 11 insertions, 7 deletions
diff --git a/src/lib/Client/Frame.py b/src/lib/Client/Frame.py index 0c13c0e76..e79c4f410 100644 --- a/src/lib/Client/Frame.py +++ b/src/lib/Client/Frame.py @@ -139,9 +139,10 @@ class Frame: mods = self.modified mbundles = [struct for struct in self.config if struct.tag == 'Bundle' and \ [mod for mod in mods if mod in struct]] - self.logger.info("The Following Bundles have been modifed:") - self.logger.info([mbun.get('name') for mbun in mbundles]) - self.logger.info("") + if mbundles: + self.logger.info("The Following Bundles have been modifed:") + self.logger.info([mbun.get('name') for mbun in mbundles]) + self.logger.info("") tbm = [(t, b) for t in self.tools for b in mbundles] for tool, bundle in tbm: try: @@ -196,8 +197,9 @@ class Frame: self.times['install'] = time.time() self.Remove() self.times['remove'] = time.time() - self.ReInventory() - self.times['reinventory'] = time.time() + if self.modified: + self.ReInventory() + self.times['reinventory'] = time.time() self.times['finished'] = time.time() self.CondDisplayState('final') diff --git a/src/lib/Client/Tools/APT.py b/src/lib/Client/Tools/APT.py index 308f157d8..a3b56a154 100644 --- a/src/lib/Client/Tools/APT.py +++ b/src/lib/Client/Tools/APT.py @@ -86,7 +86,8 @@ class APT(Bcfg2.Client.Tools.PkgTool): if len(packages) > 0: self.logger.info('Removing packages:') self.logger.info(pkgnames) - self.cmd.run("apt-get remove -y --force-yes %s" % pkgnames) + if self.cmd.run("apt-get remove -y --force-yes %s" % pkgnames)[0] == 0: + self.modified += packages self.RefreshPackages() self.extra = self.FindExtraPackages() diff --git a/src/lib/Client/Tools/RPM.py b/src/lib/Client/Tools/RPM.py index efdd05a90..31a771c29 100644 --- a/src/lib/Client/Tools/RPM.py +++ b/src/lib/Client/Tools/RPM.py @@ -79,6 +79,7 @@ class RPM(Bcfg2.Client.Tools.PkgTool): pkgnames = [pkg.get('name') for pkg in packages] if len(pkgnames) > 0: self.logger.info("Removing packages: %s" % pkgnames) - self.cmd.run("rpm --quiet -e --allmatches %s" % " ".join(pkgnames)) + if self.cmd.run("rpm --quiet -e --allmatches %s" % " ".join(pkgnames))[0] == 0: + self.modified += packages self.RefreshPackages() self.extra = self.FindExtraPackages() |