[cfe-commits] r142721 - /cfe/trunk/test/Driver/debug.c

NAKAMURA Takumi geek4civic at gmail.com
Sat Oct 22 04:28:32 PDT 2011


Author: chapuni
Date: Sat Oct 22 06:28:32 2011
New Revision: 142721

URL: http://llvm.org/viewvc/llvm-project?rev=142721&view=rev
Log:
test/Driver/debug.c: Mark this as "REQUIRES: shell".

Former RUN line does not make sense on Win32 hosts.
Win32 hosts would not be expected to set appropriate $PWD.

Latter RUN line might be made valid if PathV2::is_absolute("/foo") would be true. Unfortunately, "/foo" should not be treated as absolute path on Win32.

FYI, on mingw32 with MSYS bash (it has 'shell' feature);
Former) $PWD is set as "X:/hogehoge/test/Driver"
Latter) PWD=/foo sets "X:/root/to/msys/foo" to $PWD.

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

Modified: cfe/trunk/test/Driver/debug.c
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/debug.c?rev=142721&r1=142720&r2=142721&view=diff
==============================================================================
--- cfe/trunk/test/Driver/debug.c (original)
+++ cfe/trunk/test/Driver/debug.c Sat Oct 22 06:28:32 2011
@@ -7,4 +7,5 @@
 // "PWD=/foo gcc" wouldn't necessarily work. You would need to pick a different
 // path to the same directory (try a symlink).
 
-// XFAIL: win32
+// This depends on host's behavior how $PWD would be set.
+// REQUIRES: shell





More information about the cfe-commits mailing list