[llvm-commits] [llvm] r85028 - /llvm/trunk/utils/lit/TestRunner.py

[Daniel Dunbar]

Author: ddunbar
Date: Sat Oct 24 20:37:26 2009
New Revision: 85028

URL: http://llvm.org/viewvc/llvm-project?rev=85028&view=rev
Log:
lit: Allow use of /dev/null in redirects on Windows (replace by a temporary
file).

Modified:
    llvm/trunk/utils/lit/TestRunner.py

Modified: llvm/trunk/utils/lit/TestRunner.py
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/TestRunner.py?rev=85028&r1=85027&r2=85028&view=diff

==============================================================================
--- llvm/trunk/utils/lit/TestRunner.py (original)
+++ llvm/trunk/utils/lit/TestRunner.py Sat Oct 24 20:37:26 2009
@@ -15,6 +15,10 @@
 
 # Don't use close_fds on Windows.
 kUseCloseFDs = platform.system() != 'Windows'
+
+# Use temporary files to replace /dev/null on Windows.
+kAvoidDevNull = platform.system() == 'Windows'
+
 def executeCommand(command, cwd=None, env=None):
     p = subprocess.Popen(command, cwd=cwd,
                          stdin=subprocess.PIPE,
@@ -104,7 +108,10 @@
                 result = subprocess.PIPE
             else:
                 if r[2] is None:
-                    r[2] = open(r[0], r[1])
+                    if kAvoidDevNull and r[0] == '/dev/null':
+                        r[2] = tempfile.TemporaryFile(mode=r[1])
+                    else:
+                        r[2] = open(r[0], r[1])
                 result = r[2]
             final_redirects.append(result)