r187737 - lit.cfg: better check for MSYS

Hans Wennborg hans at hanshq.net
Mon Aug 5 13:14:43 PDT 2013


Author: hans
Date: Mon Aug  5 15:14:43 2013
New Revision: 187737

URL: http://llvm.org/viewvc/llvm-project?rev=187737&view=rev
Log:
lit.cfg: better check for MSYS

When running the tests under Cygwin using non-Cygwin python,
the platform would be Windows and there would be bash on the path,
so this check for MSYS would not work correctly.

Modified:
    cfe/trunk/test/lit.cfg

Modified: cfe/trunk/test/lit.cfg
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/lit.cfg?rev=187737&r1=187736&r2=187737&view=diff
==============================================================================
--- cfe/trunk/test/lit.cfg (original)
+++ cfe/trunk/test/lit.cfg Mon Aug  5 15:14:43 2013
@@ -245,7 +245,7 @@ if execute_external:
     config.available_features.add('shell')
 
 # Exclude MSYS due to transforming '/' to 'X:/mingwroot/'.
-if not platform.system() in ['Windows'] or lit.getBashPath() == '':
+if not platform.system() in ['Windows'] or not execute_external:
     config.available_features.add('shell-preserves-root')
 
 # ANSI escape sequences in non-dumb terminal





More information about the cfe-commits mailing list