From: Chris Larson <chris_larson@mentor.com>
Date: Thu, 30 Dec 2010 06:44:21 +0000 (-0700)
Subject: process: fix handling of the input argument
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=5ec4ca7e45bdf6d259503fc67155395e89ba6329;p=thirdparty%2Fopenembedded%2Fopenembedded-core-contrib.git

process: fix handling of the input argument

When using a logfile, we weren't sending input to the child process.

Signed-off-by: Chris Larson <chris_larson@mentor.com>
---

diff --git a/lib/bb/process.py b/lib/bb/process.py
index dc97e3f72f7..0e19cbe8de4 100644
--- a/lib/bb/process.py
+++ b/lib/bb/process.py
@@ -63,23 +63,26 @@ class Popen(subprocess.Popen):
         subprocess.Popen.__init__(self, *args, **options)
 
 def _logged_communicate(pipe, log, input):
+    if pipe.stdin:
+        if input is not None:
+            pipe.stdin.write(input)
+        pipe.stdin.close()
+
     bufsize = 512
-    hasoutput = pipe.stdout is not None or pipe.stderr is not None
-    if hasoutput:
-        outdata, errdata = [], []
-        while pipe.poll() is None:
-            if pipe.stdout is not None:
-                data = pipe.stdout.read(bufsize)
-                if data is not None:
-                    outdata.append(data)
-                    log.write(data)
-
-            if pipe.stderr is not None:
-                data = pipe.stderr.read(bufsize)
-                if data is not None:
-                    errdata.append(data)
-                    log.write(data)
-        return ''.join(outdata), ''.join(errdata)
+    outdata, errdata = [], []
+    while pipe.poll() is None:
+        if pipe.stdout is not None:
+            data = pipe.stdout.read(bufsize)
+            if data is not None:
+                outdata.append(data)
+                log.write(data)
+
+        if pipe.stderr is not None:
+            data = pipe.stderr.read(bufsize)
+            if data is not None:
+                errdata.append(data)
+                log.write(data)
+    return ''.join(outdata), ''.join(errdata)
 
 def run(cmd, input=None, **options):
     """Convenience function to run a command and return its output, raising an