From edca0b698637c3fd0a70af7e4752a46afca938d3 Mon Sep 17 00:00:00 2001 From: Narayan Desai Date: Mon, 23 Jan 2006 22:35:40 +0000 Subject: last step of repo switches git-svn-id: https://svn.mcs.anl.gov/repos/bcfg/trunk/bcfg2@1716 ce84e21b-d406-0410-9b95-82705330c041 --- src/lib/Client/Toolset.py | 22 ++++++++++++++-------- 1 file changed, 14 insertions(+), 8 deletions(-) (limited to 'src/lib/Client/Toolset.py') diff --git a/src/lib/Client/Toolset.py b/src/lib/Client/Toolset.py index b358ac92c..9cf9ad4b1 100644 --- a/src/lib/Client/Toolset.py +++ b/src/lib/Client/Toolset.py @@ -4,7 +4,7 @@ __revision__ = '$Revision$' from binascii import a2b_base64 from copy import deepcopy from grp import getgrgid, getgrnam -from os import chown, chmod, lstat, mkdir, stat, system, unlink, rename, readlink, symlink +from os import chown, chmod, lstat, mkdir, stat, unlink, rename, readlink, symlink from pwd import getpwuid, getpwnam from stat import S_ISVTX, S_ISGID, S_ISUID, S_IXUSR, S_IWUSR, S_IRUSR, S_IXGRP from stat import S_IWGRP, S_IRGRP, S_IXOTH, S_IWOTH, S_IROTH, ST_MODE, S_ISDIR @@ -87,6 +87,7 @@ class Toolset(object): pass def get_height_width(self): + '''Get Terminal information''' try: import termios, struct, fcntl height, width = struct.unpack('hhhh', @@ -99,6 +100,7 @@ class Toolset(object): return 25, 80 def FormattedCondPrint(self, state, items): + '''Formatted conditional print''' items.sort() screenWidth = self.width - len("%s[%s]:" % (self.__name__, state)) columnWidth = 1 @@ -121,11 +123,13 @@ class Toolset(object): self.CondPrint(state, lineText.rstrip()) def CondDisplayList(self, state, title, items): + '''Conditionally print a list of data''' self.CondPrint(state, title) self.FormattedCondPrint(state, items) self.CondPrint(state, '') def CondDisplayState(self, state, phase): + '''Conditionally print tracing information''' self.CondPrint(state, 'Phase: %s' % phase) self.CondPrint(state, 'Correct entries:\t%d' % self.states.values().count(True)) @@ -625,17 +629,19 @@ class Toolset(object): self.CondPrint("debug", "Installing packages: :%s:" % pkgargs) self.CondPrint("debug", "Running command ::%s::" % (pkgtool[0] % pkgargs)) - (cmdrc, cmdoutput) = self.saferun(pkgtool[0] % pkgargs) + cmdrc = self.saferun(pkgtool[0] % pkgargs)[0] if cmdrc == 0: self.CondPrint('verbose', "Single Pass Succeded") # set all package states to true and flush workqueues pkgnames = [pkg.get('name') for pkg in pkglist] for entry in [entry for entry in self.states.keys() - if entry.tag == 'Package' and entry.get('type') == pkgtype and entry.get('name') in pkgnames]: + if entry.tag == 'Package' and entry.get('type') == pkgtype + and entry.get('name') in pkgnames]: self.CondPrint('debug', 'Setting state to true for pkg %s' % (entry.get('name'))) self.states[entry] = True - [self.pkgwork[listname].remove(entry) for listname in ['add', 'update'] if self.pkgwork[listname].count(entry)] + [self.pkgwork[listname].remove(entry) for listname in ['add', 'update'] + if self.pkgwork[listname].count(entry)] self.Refresh() else: self.CondPrint("verbose", "Single Pass Failed") @@ -649,10 +655,10 @@ class Toolset(object): else: self.CondPrint("verbose", "Installing pkg %s version %s" % (pkg.get('name'), pkg.get('version'))) - (cmdrc, cmdoutput) = self.saferun(pkgtool[0] % - (pkgtool[1][0] % - tuple([pkg.get(field) for field in pkgtool[1][1]]))) - if cmdrc == 0: + cmdrc = self.saferun(pkgtool[0] % + (pkgtool[1][0] % + tuple([pkg.get(field) for field in pkgtool[1][1]]))) + if cmdrc[0] == 0: self.states[pkg] = True else: self.CondPrint('verbose', "Failed to install package %s" % (pkg.get('name'))) -- cgit v1.2.3-1-g7c22