<div dir="ltr">Hi Reid,<div><br></div><div>LGTM, we definitely should prefer the internal runner by default, and having an override to force the behavior seems reasonable.</div><div><br></div><div style> - Daniel</div></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Apr 5, 2013 at 8:20 AM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
    - Conditionalize execute_external on an environment variable.<br>
<br>
Hi chapuni,<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D559" target="_blank">http://llvm-reviews.chandlerc.com/D559</a><br>
<br>
CHANGE SINCE LAST DIFF<br>
  <a href="http://llvm-reviews.chandlerc.com/D559?vs=1340&id=1525#toc" target="_blank">http://llvm-reviews.chandlerc.com/D559?vs=1340&id=1525#toc</a><br>
<div class="im"><br>
Files:<br>
  test/lit.cfg<br>
<br>
Index: test/lit.cfg<br>
===================================================================<br>
--- test/lit.cfg<br>
+++ test/lit.cfg<br>
</div>@@ -22,9 +22,18 @@<br>
                                      config.environment['PATH']))<br>
         config.environment['PATH'] = path<br>
<br>
+# Choose between lit's internal shell pipeline runner and a real shell.  If<br>
+# LIT_USE_INTERNAL_SHELL is in the environment, we use that as an override.<br>
+use_lit_shell = os.environ.get("LIT_USE_INTERNAL_SHELL")<br>
+if use_lit_shell:<br>
+    # 0 is external, "" is default, and everything else is internal.<br>
+    execute_external = (use_lit_shell == "0")<br>
+else:<br>
+    # Otherwise we default to internal on Windows and external elsewhere, as<br>
+    # bash on Windows is usually very slow.<br>
+    execute_external = (not sys.platform in ['win32'])<br>
<div class="im">+<br>
 # testFormat: The test format to use to interpret tests.<br>
-execute_external = (not sys.platform in ['win32']<br>
-                    or lit.getBashPath() not in [None, ""])<br>
</div><div class="im"> config.test_format = lit.formats.ShTest(execute_external)<br>
<br>
 # To ignore test output on stderr so it doesn't trigger failures uncomment this:<br>
</div>@@ -240,7 +249,7 @@<br>
<div class="HOEnZb"><div class="h5"> ### Features<br>
<br>
 # Shell execution<br>
-if sys.platform not in ['win32'] or lit.getBashPath() != '':<br>
+if execute_external:<br>
     config.available_features.add('shell')<br>
<br>
 # Loadable module<br>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>