<div dir="ltr">One other thing to look at here is execute_external, which controls whether or not we're actually using bash from Cygwin.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 29, 2013 at 10:44 AM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I use the system's Python, maybe that's why we get different results:<br>
<br>
$ /cygdrive/c/python_26_amd64/files/python bin/llvm-lit -v<br>
../tools/clang/test/Driver/cl.c<br>
-- Testing: 1 tests, 1 threads --<br>
UNSUPPORTED: Clang :: Driver/cl.c (1 of 1)<br>
Testing Time: 0.00s<br>
  Unsupported Tests  : 1<br>
llvm-lit: lit.cfg:184: note: using clang:<br>
'D:/src/llvm/build.debug/bin/./clang.EXE'<br>
<br>
In my case, lit.getBashPath() returns "C:\src\cygwin\bin\bash.EXE".<br>
<br>
Do you think my patch is OK?<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
On Mon, Jul 29, 2013 at 10:34 AM, NAKAMURA Takumi <<a href="mailto:geek4civic@gmail.com">geek4civic@gmail.com</a>> wrote:<br>
> Hans, I am certain it would be less useful on cygwin.<br>
><br>
> Python 2.6.8 (unknown, Jun  9 2012, 11:30:32)<br>
>>>> import platform<br>
>>>> platform.system()<br>
> 'CYGWIN_NT-6.1-WOW64'<br>
><br>
> I can see "shell-preserves-root" is enabled on cygwin.<br>
><br>
> PASS: Clang :: Driver/cl.c (2638 of 6275)<br>
><br>
> Which version of python are you using on cygwin?<br>
><br>
><br>
> 2013/7/30 Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>>:<br>
>> Hi,<br>
>><br>
>> The attached patch tightens the "shell-preserves-root" feature check<br>
>> in lit.cfg. Previously, the check would exclude this feature also on<br>
>> Cygwin, but as the comment suggests, I think it should only check for<br>
>> MSYS.<br>
>><br>
>> With this patch, the number of unsupported tests on Cygwin go from 35 to 32.<br>
>><br>
>> I was not able to test this on MSYS. Takumi, could you give it a try?<br>
>><br>
>> Thanks,<br>
>> Hans<br>
</div></div></blockquote></div><br></div>