r187330 - Try single quotes to fix the msys bot:
Reid Kleckner
rnk at google.com
Sun Jul 28 10:38:18 PDT 2013
The msys shell is applying these transformations:
http://www.mingw.org/wiki/Posix_path_conversion
It looks like this commit didn't fix the test, and that page doesn't show
any way to avoid this conversion.
On Sun, Jul 28, 2013 at 1:13 PM, Rafael Espindola <
rafael.espindola at gmail.com> wrote:
> Author: rafael
> Date: Sun Jul 28 12:13:24 2013
> New Revision: 187330
>
> URL: http://llvm.org/viewvc/llvm-project?rev=187330&view=rev
> Log:
> Try single quotes to fix the msys bot:
>
> It was still failing with double quotes:
>
>
> http://bb.pgr.jp/builders/clang-i686-msys/builds/698/steps/test_clang/logs/Clang%20%3A%3A%20Driver__cl.c
>
> 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=187330&r1=187329&r2=187330&view=diff
>
> ==============================================================================
> --- cfe/trunk/test/Driver/cl.c (original)
> +++ cfe/trunk/test/Driver/cl.c Sun Jul 28 12:13:24 2013
> @@ -1,19 +1,18 @@
> -// 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 if such a file or directory exists.
> +// Note: we have to quote the options with a /, otherwise some shells
> will try
> +// to expand it and make test fail.
>
> // 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: {{/[?]}}
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130728/c236340a/attachment.html>
More information about the cfe-commits
mailing list