r187366 - test/Driver/cl.c, remove quotes around /help

Hans Wennborg hans at hanshq.net
Mon Jul 29 11:40:38 PDT 2013


Author: hans
Date: Mon Jul 29 13:40:38 2013
New Revision: 187366

URL: http://llvm.org/viewvc/llvm-project?rev=187366&view=rev
Log:
test/Driver/cl.c, remove quotes around /help

The quotes (from r187330) didn't really help here, the trick was to disable
the test on MSYS builds. This removes those quotes, changes back the comment
to explain why /? has to be quoted specifically, and moves the REQUIRES
line to the top of the file because that's important.

Modified:
    cfe/trunk/test/Driver/cl.c

Modified: cfe/trunk/test/Driver/cl.c
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/cl.c?rev=187366&r1=187365&r2=187366&view=diff
==============================================================================
--- cfe/trunk/test/Driver/cl.c (original)
+++ cfe/trunk/test/Driver/cl.c Mon Jul 29 13:40:38 2013
@@ -1,18 +1,22 @@
-// Note: we have to quote the options with a /, otherwise some shells will try
-// to expand it and make test fail.
+// Don't attempt slash switches on msys bash.
+// REQUIRES: shell-preserves-root
+
+// Note: we have to quote the /? option, otherwise some shells will try to
+// expand the ? into a one-letter filename in the root directory, and make
+// the test fail is such a file or directory exists.
 
 // Check that clang-cl options are not available by default.
 // RUN: %clang -help | FileCheck %s -check-prefix=DEFAULT
 // DEFAULT-NOT: CL.EXE COMPATIBILITY OPTIONS
 // DEFAULT-NOT: {{/[?]}}
 // DEFAULT-NOT: /help
-// RUN: not %clang '/?'
+// RUN: not %clang "/?"
 // RUN: not %clang -?
-// RUN: not %clang '/help'
+// RUN: not %clang /help
 
-// Check that '/?' and '/help' are available as clang-cl options.
-// RUN: %clang_cl '/?' | FileCheck %s -check-prefix=CL
-// RUN: %clang_cl '/help' | FileCheck %s -check-prefix=CL
+// Check that /? and /help are available as clang-cl options.
+// RUN: %clang_cl "/?" | FileCheck %s -check-prefix=CL
+// RUN: %clang_cl /help | FileCheck %s -check-prefix=CL
 // RUN: %clang_cl -help | FileCheck %s -check-prefix=CL
 // CL: CL.EXE COMPATIBILITY OPTIONS
 // CL: {{/[?]}}
@@ -22,6 +26,3 @@
 // are not available in clang-cl.
 // DEFAULT: -fapple-kext
 // CL-NOT: -fapple-kext
-
-// Don't attempt slash switches on msys bash.
-// REQUIRES: shell-preserves-root





More information about the cfe-commits mailing list