<div dir="ltr">I'd just go for it.  That patch looks fine and should have very low risk.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 5, 2013 at 10:40 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"><div class="im">On Tue, Jul 30, 2013 at 3:56 PM, Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br>

> On Mon, Jul 29, 2013 at 6:07 PM, Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br>
>> On Mon, Jul 29, 2013 at 3:28 PM, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>
>>> bash -help is pretty fast to start, but it also seems pretty heavyweight for<br>
>>> lit.cfg.  It's probably better to check not execute_external like this:<br>
>>><br>
>>> $ git diff test/lit.cfg<br>
>>> diff --git a/test/lit.cfg b/test/lit.cfg<br>
>>> index a5bb350..f2f16fb 100644<br>
>>> --- a/test/lit.cfg<br>
>>> +++ b/test/lit.cfg<br>
>>> @@ -245,7 +245,7 @@ if execute_external:<br>
>>>      config.available_features.add('shell')<br>
>>><br>
>>>  # Exclude MSYS due to transforming '/' to 'X:/mingwroot/'.<br>
>>> -if not platform.system() in ['Windows'] or lit.getBashPath() == '':<br>
>>> +if not platform.system() in ['Windows'] or not execute_external:<br>
>>>      config.available_features.add('shell-preserves-root')<br>
>><br>
>> That sounds good to me. Takumi, what do you think about this change? I<br>
>> see we do (almost) the same check earlier in the file, in<br>
>> getClangBuiltinIncludeDir().<br>
><br>
> Ping?<br>
<br>
</div>Ping?<br>
</blockquote></div><br></div>