diff options
author | Narayan Desai <desai@mcs.anl.gov> | 2006-12-14 02:30:44 +0000 |
---|---|---|
committer | Narayan Desai <desai@mcs.anl.gov> | 2006-12-14 02:30:44 +0000 |
commit | 5ba90c46c7372bc7979ae875c712d4d211c2820e (patch) | |
tree | d5260116580df22870a222dca99de0da7490de99 /src/lib/Client/Tools/__init__.py | |
parent | 9fba07d3f602914fd8710f9279110de3cf6b71c4 (diff) | |
download | bcfg2-5ba90c46c7372bc7979ae875c712d4d211c2820e.tar.gz bcfg2-5ba90c46c7372bc7979ae875c712d4d211c2820e.tar.bz2 bcfg2-5ba90c46c7372bc7979ae875c712d4d211c2820e.zip |
Fix SMF problems on solaris
* SMF extra package location rewrite
* SubCMD implementation change
git-svn-id: https://svn.mcs.anl.gov/repos/bcfg/trunk/bcfg2@2593 ce84e21b-d406-0410-9b95-82705330c041
Diffstat (limited to 'src/lib/Client/Tools/__init__.py')
-rw-r--r-- | src/lib/Client/Tools/__init__.py | 21 |
1 files changed, 8 insertions, 13 deletions
diff --git a/src/lib/Client/Tools/__init__.py b/src/lib/Client/Tools/__init__.py index 9b7acee02..56d9aba8b 100644 --- a/src/lib/Client/Tools/__init__.py +++ b/src/lib/Client/Tools/__init__.py @@ -38,21 +38,16 @@ class executor: self.logger.debug('> %s' % command) runpipe = readonlypipe(command, bufsize=16384) - output = '' - cmdstat = -1 - while cmdstat == -1: - runpipe.fromchild.flush() - moreOutput = runpipe.fromchild.readline() - if len(moreOutput) > 0: - self.logger.debug('< %s' % moreOutput[:-1]) - output += moreOutput - cmdstat = runpipe.poll() - for line in runpipe.fromchild.readlines(): + output = [] + runpipe.fromchild.flush() + line = runpipe.fromchild.readline() + while line: if len(line) > 0: self.logger.debug('< %s' % line[:-1]) - output += line - - return (cmdstat, [line for line in output.split('\n') if line]) + output.append(line[:-1]) + line = runpipe.fromchild.readline() + cmdstat = runpipe.poll() + return (cmdstat, output) class Tool: '''All tools subclass this. It defines all interfaces that need to be defined''' |