Should the substitution just be surrounded by quotes automatically so we don’t have this problem?<br><div class="gmail_quote"><div dir="ltr">On Thu, Aug 2, 2018 at 4:19 PM Stella Stamenova 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">stella.stamenova created this revision.<br>
stella.stamenova added reviewers: asmith, zturner.<br>
Herald added subscribers: llvm-commits, delcypher.<br>
<br>
The issue with these tests is that they use python, but the path to python on Windows can contain spaces. To make the tests always work, pass the path surrounded by quotes.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D50206" rel="noreferrer" target="_blank">https://reviews.llvm.org/D50206</a><br>
<br>
Files:<br>
  utils/lit/tests/boolean-parsing.py<br>
  utils/lit/tests/discovery.py<br>
  utils/lit/tests/lit.cfg<br>
  utils/lit/tests/shell-parsing.py<br>
  utils/lit/tests/unittest-adaptor.py<br>
<br>
<br>
Index: utils/lit/tests/unittest-adaptor.py<br>
===================================================================<br>
--- utils/lit/tests/unittest-adaptor.py<br>
+++ utils/lit/tests/unittest-adaptor.py<br>
@@ -1,6 +1,6 @@<br>
 # Check the lit adaption to run under unittest.<br>
 #<br>
-# RUN: %{python} %s %{inputs}/unittest-adaptor 2> %t.err<br>
+# RUN: '%{python}' %s %{inputs}/unittest-adaptor 2> %t.err<br>
 # RUN: FileCheck < %t.err %s<br>
 #<br>
 # CHECK-DAG: unittest-adaptor :: test-two.txt ... FAIL<br>
Index: utils/lit/tests/shell-parsing.py<br>
===================================================================<br>
--- utils/lit/tests/shell-parsing.py<br>
+++ utils/lit/tests/shell-parsing.py<br>
@@ -1,3 +1,3 @@<br>
 # Just run the ShUtil unit tests.<br>
 #<br>
-# RUN: %{python} -m lit.ShUtil<br>
+# RUN: '%{python}' -m lit.ShUtil<br>
Index: utils/lit/tests/lit.cfg<br>
===================================================================<br>
--- utils/lit/tests/lit.cfg<br>
+++ utils/lit/tests/lit.cfg<br>
@@ -38,7 +38,7 @@<br>
 config.substitutions.append(('%{src_root}', src_root))<br>
 config.substitutions.append(('%{inputs}', os.path.join(<br>
             src_root, 'tests', 'Inputs')))<br>
-config.substitutions.append(('%{lit}', "%%{python} %s" % (<br>
+config.substitutions.append(('%{lit}', "\'%%{python}\' %s" % (<br>
             os.path.join(lit_path, 'lit.py'),)))<br>
 config.substitutions.append(('%{python}', sys.executable))<br>
<br>
Index: utils/lit/tests/discovery.py<br>
===================================================================<br>
--- utils/lit/tests/discovery.py<br>
+++ utils/lit/tests/discovery.py<br>
@@ -26,7 +26,7 @@<br>
 # CHECK-BASIC-OUT: top-level-suite :: test-two<br>
<br>
 # Check discovery when providing the special builtin 'config_map'<br>
-# RUN: %{python} %{inputs}/config-map-discovery/driver.py \<br>
+# RUN: '%{python}' %{inputs}/config-map-discovery/driver.py \<br>
 # RUN:           %{inputs}/config-map-discovery/main-config/lit.cfg \<br>
 # RUN:           %{inputs}/config-map-discovery/lit.alt.cfg \<br>
 # RUN:           --single-process --debug --show-tests --show-suites > %t.out 2> %t.err<br>
Index: utils/lit/tests/boolean-parsing.py<br>
===================================================================<br>
--- utils/lit/tests/boolean-parsing.py<br>
+++ utils/lit/tests/boolean-parsing.py<br>
@@ -1,4 +1,4 @@<br>
 # Test the boolean expression parser<br>
 # used for REQUIRES and UNSUPPORTED and XFAIL<br>
<br>
-# RUN: %{python} -m lit.BooleanExpression<br>
+# RUN: '%{python}' -m lit.BooleanExpression<br>
<br>
<br>
</blockquote></div>