From 60f7756626271b46ee7d32a80ba1e78b337d6f91 Mon Sep 17 00:00:00 2001 From: Peter Scheibel Date: Wed, 25 Nov 2015 12:28:43 -0800 Subject: Executable.__call__ original semantics were to avoid redirecting process output to the output stream if return_output was set. This branch undid that and this commit restores those semantics. --- lib/spack/spack/util/executable.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/spack/spack/util/executable.py b/lib/spack/spack/util/executable.py index 6848a7e2a5..6088d4514e 100644 --- a/lib/spack/spack/util/executable.py +++ b/lib/spack/spack/util/executable.py @@ -100,7 +100,8 @@ class Executable(object): out, err = proc.communicate() self.returncode = proc.returncode - output.write(out) + if not return_output: + output.write(out) error.write(err) rc = proc.returncode -- cgit v1.2.3-70-g09d2