[ARVADOS] updated: f09b538f62240646f7715543f66ecf521241fd78

[git at public.curoverse.com]

Summary of changes:
 sdk/python/arvados/util.py | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

       via  f09b538f62240646f7715543f66ecf521241fd78 (commit)
      from  e4382e25d68a70a107baeb257e1915972759334c (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit f09b538f62240646f7715543f66ecf521241fd78
Author: Tom Clegg <tom at curoverse.com>
Date:   Mon Aug 25 20:59:37 2014 -0400

    Do not blow up when stdout!=subprocess.PIPE. no issue #

diff --git a/sdk/python/arvados/util.py b/sdk/python/arvados/util.py
index d5ea18b..22a7427 100644
--- a/sdk/python/arvados/util.py
+++ b/sdk/python/arvados/util.py
@@ -30,10 +30,7 @@ def run_command(execargs, **kwargs):
     kwargs.setdefault('close_fds', True)
     kwargs.setdefault('shell', False)
     p = subprocess.Popen(execargs, **kwargs)
-    if kwargs['stdout'] == subprocess.PIPE:
-        stdoutdata, stderrdata = p.communicate(None)
-    else:
-        p.wait()
+    stdoutdata, stderrdata = p.communicate(None)
     if p.returncode != 0:
         raise errors.CommandFailedError(
             "run_command %s exit %d:\n%s" %

-----------------------------------------------------------------------


hooks/post-receive
--