Lgtm<br><div class="gmail_quote"><div dir="ltr">On Mon, Sep 10, 2018 at 11:16 AM 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, delcypher, zturner.<br>
Herald added a subscriber: llvm-commits.<br>
<br>
This is the only test that is still failing on Windows - or rather, it is expected to fail on the bots, but passes on the new bot that we're preparing causing a failure, so I'm going to disable it. Since the test has rarely, if ever, passed on the bots, this should have the same effect and it will unblock the creation of the new bot.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D51871" rel="noreferrer" target="_blank">https://reviews.llvm.org/D51871</a><br>
<br>
Files:<br>
  utils/lit/tests/Inputs/shtest-timeout/lit.cfg<br>
  utils/lit/tests/shtest-timeout.py<br>
<br>
<br>
Index: utils/lit/tests/shtest-timeout.py<br>
===================================================================<br>
--- utils/lit/tests/shtest-timeout.py<br>
+++ utils/lit/tests/shtest-timeout.py<br>
@@ -1,7 +1,7 @@<br>
 # REQUIRES: python-psutil<br>
<br>
-# PR33944<br>
-# XFAIL: windows<br>
+# <a href="http://llvm.org/PR33944" rel="noreferrer" target="_blank">llvm.org/PR33944</a><br>
+# REQUIRES: nowindows<br>
<br>
 # FIXME: This test is fragile because it relies on time which can<br>
 # be affected by system performance. In particular we are currently<br>
Index: utils/lit/tests/Inputs/shtest-timeout/lit.cfg<br>
===================================================================<br>
--- utils/lit/tests/Inputs/shtest-timeout/lit.cfg<br>
+++ utils/lit/tests/Inputs/shtest-timeout/lit.cfg<br>
@@ -13,9 +13,7 @@<br>
     externalShell = False<br>
 else:<br>
     lit_config.note('Using external shell')<br>
-    externalShell = (not sys.platform in ['win32'])<br>
-    if not externalShell:<br>
-        lit_config.note('External shell requested, but using internal shell')<br>
+    externalShell = True<br>
<br>
 configSetTimeout = lit_config.params.get('set_timeout', '0')<br>
<br>
<br>
<br>
</blockquote></div>