[PATCH] D39278: [lit] Respect LLVM_LIT_TOOLS_DIR when looking for 'tar' on Windows.

Zhihao Yuan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 25 03:10:10 PDT 2017


lichray added inline comments.


================
Comment at: test/lit.cfg.py:90
+        [tar_executable, '--version'], stdout=subprocess.PIPE, env={'LANG': 'C'})
+    if 'GNU tar' in tar_version.stdout.read().decode():
+        config.available_features.add('gnutar')
----------------
How about just read with `tar_version, _ = communicate()` and remove the '.wait()'?


Repository:
  rL LLVM

https://reviews.llvm.org/D39278





More information about the llvm-commits mailing list