- explicitly make argv a list in execWith{Capture,Redirect}

2 files changed, 6 insertions, 0 deletions

diff --git a/ChangeLog b/ChangeLog
index 346327255..8641c8e2d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2007-01-15  Peter Jones  <pjones@redhat.com>
+
+	* iutil.py (execWith{Capture,Redirect}: force argv a list, not a tuple
+
 2007-01-11  Jeremy Katz  <katzj@redhat.com>
 
 	* yuminstall.py (AnacondaYum._run): Fix so that we properly show
diff --git a/iutil.py b/iutil.py
index 02eccfcb7..0a64cf2a5 100644
--- a/iutil.py
+++ b/iutil.py
@@ -32,6 +32,7 @@ def execWithRedirect(command, argv, stdin = 0, stdout = 1, stderr = 2,
         if not searchPath and not os.access (command, os.X_OK):
             raise RuntimeError, command + " can not be run"
 
+    argv = list(argv)
     if type(stdin) == type("string"):
         if os.access(stdin, os.R_OK):
             stdin = open(stdin)
@@ -55,6 +56,7 @@ def execWithCapture(command, argv, stdin = 0, stderr = 2, root='/'):
     def chroot():
         os.chroot(root)
 
+    argv = list(argv)
     if type(stdin) == type("string"):
         if os.access(stdin, os.R_OK):
             stdin = open(stdin)