<div dir="ltr">On Mon, Jun 17, 2013 at 2:44 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</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 17 June 2013 06:48, Manuel Klimek <<a href="mailto:klimek@google.com">klimek@google.com</a>> wrote:<br>

> Author: klimek<br>
> Date: Mon Jun 17 05:48:34 2013<br>
> New Revision: 184074<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=184074&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=184074&view=rev</a><br>
> Log:<br>
> Fix incorrectly finding 'executable' directories instead of files.<br>
><br>
> This broke for example the 'not' utility, if a directory called<br>
> 'FileCheck' is executable and in the path before the actual 'FileCheck'.<br>
><br>
> This patch steals the implementation of the "old" PathV1 canExecute<br>
> implementation:<br>
> - checks for R_OK (file readable): this is necessary for executing<br>
>   scripts; we should not regress here unless we have good reasons<br>
<br>
</div>This is ok.<br>
<div class="im"><br>
> - checks for S_ISREG; if we want to get rid of this, we'd need to<br>
>   change all callers who already made the assumption when depending<br>
>   on Path V1.<br>
<br>
</div>Sigh, I was hopping to avoid the cost, but yes, it is better to have it for now.<br>
<br>
I will try to add testcases.<br></blockquote><div><br></div><div style>Cool, thanks! :)</div><div style> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks,<br>
Rafael<br>
</blockquote></div><br></div></div>