From: Zachary Turner <zturner@google.com>
Date: Sat, 7 Feb 2015 00:14:55 +0000 (+0000)
Subject: Dont' use close_fds = True on Windows.
X-Git-Tag: llvmorg-3.7.0-rc1~12782
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=dc494d5303e01c36237de42ec09077b03b58fa88;p=platform%2Fupstream%2Fllvm.git

Dont' use close_fds = True on Windows.

If you do, the test runner will fail immediately with the error:

close_fds is not supported on Windows platforms if you redirect
stdin/stdout/stderr.

llvm-svn: 228472
---

diff --git a/lldb/test/dosep.py b/lldb/test/dosep.py
index e397eab..dd43761 100755
--- a/lldb/test/dosep.py
+++ b/lldb/test/dosep.py
@@ -55,11 +55,18 @@ eTimedOut, ePassed, eFailed = 124, 0, 1
 
 def call_with_timeout(command, timeout):
     """Run command with a timeout if possible."""
-    if timeout_command and timeout != "0":
-        return subprocess.call([timeout_command, timeout] + command,
-                               stdin=subprocess.PIPE, close_fds=True)
-    return (ePassed if subprocess.call(command, stdin=subprocess.PIPE, close_fds=True) == 0
-            else eFailed)
+    if os.name != "nt":
+        if timeout_command and timeout != "0":
+            return subprocess.call([timeout_command, timeout] + command,
+                                   stdin=subprocess.PIPE, close_fds=True)
+        return (ePassed if subprocess.call(command, stdin=subprocess.PIPE, close_fds=True) == 0
+                else eFailed)
+    else:
+        if timeout_command and timeout != "0":
+            return subprocess.call([timeout_command, timeout] + command,
+                                   stdin=subprocess.PIPE)
+        return (ePassed if subprocess.call(command, stdin=subprocess.PIPE) == 0
+                else eFailed)
 
 def process_dir(root, files, test_root, dotest_options):
     """Examine a directory for tests, and invoke any found within it."""