<p dir="ltr"><br>
On Aug 2, 2015 00:45, "Jordan Justen" <<a href="mailto:jordan.l.justen@intel.com">jordan.l.justen@intel.com</a>> wrote:<br>
><br>
> If xz is present, and the piglit command's output is redirected, then<br>
> xz will think its output is redirected. This will cause xz to try to<br>
> read data from stdin to compress.<br>
><br>
> Instead we can run xz with --help to cause it to print help<br>
> information if the xz executable is present, and prevent xz from<br>
> trying to compress data from stdin.<br>
><br>
> Since we don't want to see 'xz --help' output from piglit, we now need<br>
> to redirect both stdout and stderr to /dev/null.<br>
><br>
> Signed-off-by: Jordan Justen <<a href="mailto:jordan.l.justen@intel.com">jordan.l.justen@intel.com</a>><br>
> Cc: Dylan Baker <<a href="mailto:baker.dylan.c@gmail.com">baker.dylan.c@gmail.com</a>><br>
> ---<br>
>  framework/backends/compression.py | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/framework/backends/compression.py b/framework/backends/compression.py<br>
> index 88bde2a..97b0f32 100644<br>
> --- a/framework/backends/compression.py<br>
> +++ b/framework/backends/compression.py<br>
> @@ -95,7 +95,7 @@ try:<br>
>  except ImportError:<br>
>      try:<br>
>          with open(os.devnull, 'w') as d:<br>
> -            subprocess.check_call(['xz'], stderr=d)<br>
> +            subprocess.check_call(['xz', '--help'], stdout=d, stderr=subprocess.STDOUT)</p>
<p dir="ltr">Based on you're commit message I don't think this sides what you want. I think you should set stderr=d to hide both stderr and stdout.</p>
<p dir="ltr">>      except subprocess.CalledProcessError as e:<br>
>          if e.returncode == 1:<br>
>              import contextlib<br>
> --<br>
> 2.1.4<br>
><br>
</p>