Revert "replace deprecated popen2 with subprocess"

[bootmanager.git] / source / utils.py

diff --git a/source/utils.py b/source/utils.py
index b6b51c8..10607cc 100644 (file)
--- a/source/utils.py
+++ b/source/utils.py
@@ -11,8 +11,7 @@
 # expected /proc/partitions format
 
 import os, sys, shutil
-import subprocess
-import shlex
+import popen2
 import socket
 import fcntl
 import string
@@ -121,7 +120,7 @@ def removedir( path ):
 
 
 
-def sysexec( cmd, log= None ):
+def sysexec( cmd, log= None, fsck = False ):
     """
     execute a system command, output the results to the logger
     if log <> None
@@ -133,21 +132,36 @@ def sysexec( cmd, log= None ):
     if VERBOSE_MODE:
         print ("sysexec >>> %s" % cmd)
 
-    try:
-        prog = subprocess.Popen(shlex.split(cmd), stdout=subprocess.PIPE, stderr=subprocess.PIPE)
-    except OSError:
+    prog = popen2.Popen4( cmd, 0 )
+    if prog is None:
         raise BootManagerException, \
-              "Unable to create instance of subprocess.Popen " \
+              "Unable to create instance of popen2.Popen4 " \
               "for command: %s" % cmd
 
-    (stdoutdata, stderrdata) = prog.communicate()
     if log is not None:
-        log.write(stdoutdata)
+        try:
+            for line in prog.fromchild:
+                log.write( line )
+        except KeyboardInterrupt:
+            raise BootManagerException, "Interrupted by user"
 
     returncode = prog.wait()
-    if returncode != 0:
-        raise BootManagerException, "Running %s failed (rc=%d)" % (cmd,returncode)
 
+    if fsck:
+       # The exit code returned by fsck is the sum of the following conditions:
+       #      0    - No errors
+       #      1    - File system errors corrected
+       #      2    - System should be rebooted
+       #      4    - File system errors left uncorrected
+       #      8    - Operational error
+       #      16   - Usage or syntax error
+       #      32   - Fsck canceled by user request
+       #      128  - Shared library error
+       if returncode != 0 and returncode != 1:
+            raise BootManagerException, "Running %s failed (rc=%d)" % (cmd,returncode)
+    else:
+        if returncode != 0:
+            raise BootManagerException, "Running %s failed (rc=%d)" % (cmd,returncode)
     prog = None
     return 1