[llvm] r328596 - Revert "[lit] Generalized /dev/null support on Windows."

[Mircea Trofin via llvm-commits]

Author: mtrofin
Date: Mon Mar 26 16:59:39 2018
New Revision: 328596

URL: http://llvm.org/viewvc/llvm-project?rev=328596&view=rev
Log:
Revert "[lit] Generalized /dev/null support on Windows."

This reverts commit ca7fdbb974384ce5a05528b22a41d46b1cc13e92.

Removed:
    llvm/trunk/utils/lit/tests/Inputs/shtest-shell/check_args.py
    llvm/trunk/utils/lit/tests/Inputs/shtest-shell/dev-null.txt
Modified:
    llvm/trunk/test/tools/llvm-profdata/invalid-profdata.test
    llvm/trunk/utils/lit/lit/TestRunner.py

Modified: llvm/trunk/test/tools/llvm-profdata/invalid-profdata.test
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-profdata/invalid-profdata.test?rev=328596&r1=328595&r2=328596&view=diff
==============================================================================
--- llvm/trunk/test/tools/llvm-profdata/invalid-profdata.test (original)
+++ llvm/trunk/test/tools/llvm-profdata/invalid-profdata.test Mon Mar 26 16:59:39 2018
@@ -20,7 +20,7 @@ RUN: echo "f2:0"
 RUN: echo "1"                                            >> %t.input
 RUN: echo ":10"                                          >> %t.input
 
-RUN: not llvm-profdata merge %t.input -text -output=/dev/null 2>&1  | FileCheck %s --check-prefix=BROKEN
+RUN: not llvm-profdata merge %t.input -text -o /dev/null 2>&1  | FileCheck %s --check-prefix=BROKEN
 BROKEN: Malformed instrumentation profile data
 
 RUN: echo ":ir"                     > %t.input

Modified: llvm/trunk/utils/lit/lit/TestRunner.py
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/lit/TestRunner.py?rev=328596&r1=328595&r2=328596&view=diff
==============================================================================
--- llvm/trunk/utils/lit/lit/TestRunner.py (original)
+++ llvm/trunk/utils/lit/lit/TestRunner.py Mon Mar 26 16:59:39 2018
@@ -36,7 +36,6 @@ kUseCloseFDs = not kIsWindows
 
 # Use temporary files to replace /dev/null on Windows.
 kAvoidDevNull = kIsWindows
-kDevNull = "/dev/null"
 
 class ShellEnvironment(object):
 
@@ -627,7 +626,7 @@ def processRedirects(cmd, stdin_source,
            raise InternalShellError(cmd, "Unsupported: glob in "
                                     "redirect expanded to multiple files")
         name = name[0]
-        if kAvoidDevNull and name == kDevNull:
+        if kAvoidDevNull and name == '/dev/null':
             fd = tempfile.TemporaryFile(mode=mode)
         elif kIsWindows and name == '/dev/tty':
             # Simulate /dev/tty on Windows.
@@ -798,11 +797,11 @@ def _executeShCmd(cmd, shenv, results, t
         # Replace uses of /dev/null with temporary files.
         if kAvoidDevNull:
             for i,arg in enumerate(args):
-                if kDevNull in arg:
+                if arg == "/dev/null":
                     f = tempfile.NamedTemporaryFile(delete=False)
                     f.close()
                     named_temp_files.append(f.name)
-                    args[i] = arg.replace(kDevNull, f.name)
+                    args[i] = f.name
 
         # Expand all glob expressions
         args = expand_glob_expressions(args, cmd_shenv.cwd)

Removed: llvm/trunk/utils/lit/tests/Inputs/shtest-shell/check_args.py
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/tests/Inputs/shtest-shell/check_args.py?rev=328595&view=auto
==============================================================================
--- llvm/trunk/utils/lit/tests/Inputs/shtest-shell/check_args.py (original)
+++ llvm/trunk/utils/lit/tests/Inputs/shtest-shell/check_args.py (removed)
@@ -1,14 +0,0 @@
-#!/usr/bin/env python
-
-import argparse
-import platform
-
-parser = argparse.ArgumentParser()
-parser.add_argument("--my_arg", "-a")
-
-args = parser.parse_args()
-
-answer = (platform.system() == "Windows" and
-          args.my_arg == "/dev/null" and "ERROR") or "OK"
-
-print(answer)

Removed: llvm/trunk/utils/lit/tests/Inputs/shtest-shell/dev-null.txt
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/tests/Inputs/shtest-shell/dev-null.txt?rev=328595&view=auto
==============================================================================
--- llvm/trunk/utils/lit/tests/Inputs/shtest-shell/dev-null.txt (original)
+++ llvm/trunk/utils/lit/tests/Inputs/shtest-shell/dev-null.txt (removed)
@@ -1,14 +0,0 @@
-# Check handling of /dev/null in command line options
-# On windows, it should be redirected to a temp file.
-#
-# RUN: "%{python}" %S/check_args.py --my_arg /dev/null | FileCheck %s --check-prefix=CHECK1
-# CHECK1: OK
-
-# RUN: "%{python}" %S/check_args.py --my_arg=/dev/null | FileCheck %s --check-prefix=CHECK2
-# CHECK2: OK
-
-# RUN: "%{python}" %S/check_args.py -a /dev/null | FileCheck %s --check-prefix=CHECK3
-# CHECK3: OK
-
-# RUN: "%{python}" %S/check_args.py -a=/dev/null | FileCheck %s --check-prefix=CHECK4
-# CHECK4: OK