Seems fine <br><div class="gmail_quote"><div dir="ltr">On Wed, Oct 25, 2017 at 4:33 AM Igor Kudrin via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ikudrin added inline comments.<br>
<br>
<br>
================<br>
Comment at: test/<a href="http://lit.cfg.py:90" rel="noreferrer" target="_blank">lit.cfg.py:90</a><br>
+        [tar_executable, '--version'], stdout=subprocess.PIPE, env={'LANG': 'C'})<br>
+    if 'GNU tar' in tar_version.stdout.read().decode():<br>
+        config.available_features.add('gnutar')<br>
----------------<br>
lichray wrote:<br>
> How about just read with `tar_version, _ = communicate()` and remove the '.wait()'?<br>
I'd redirect your idea to the author of D38977, @pcc.<br>
Anyway, this patch is only about fixing an execution failure on Windows, and nothing more.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D39278" rel="noreferrer" target="_blank">https://reviews.llvm.org/D39278</a><br>
<br>
<br>
<br>
</blockquote></div>