<div dir="ltr">On Mon, Jul 29, 2013 at 11:33 AM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Mon, Jul 29, 2013 at 11:01 AM, NAKAMURA Takumi <<a href="mailto:geek4civic@gmail.com">geek4civic@gmail.com</a>> wrote:<br>

> I suppose that system's python should be /usr/bin/python in Cygwin's world.<br>
> I didn't know Python/win32 could run Lit for cygwin tests.<br>
><br>
> What is a motivation for you to take *external* python on cygwin?<br>
<br>
</div>I build Clang outside Cygwin, in cmd.exe, because I want to build in a<br>
"pure win32" environment. But I fail at running the tests in cmd.exe,<br>
so I use Cygwin to run the tests.<br>
<br>
If I try to use Cygwin's /usr/bin/python to run the tests, it fails<br>
because of some Windows path. Probably because I didn't build in<br>
Cygwin.<br>
<br>
I guess my configuration is confusing to lit.cfg, because I have<br>
platform.system() = "Windows" and lit.getBashPath() =<br>
"C:\src\cygwin\bin\bash.EXE". But it's still not MSYS :)</blockquote><div><br></div><div>This configuration is pretty handy because it's way faster to spawn processes from native win32 python than from cygwin bash or cygwin python.</div>
<div><br></div><div>I think the odd duck that we should be checking for here is MSys, so using the MSYSTEM env var sounds good to me.</div></div></div></div>