<div class="gmail_quote">On Fri, Feb 3, 2012 at 7:20 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</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">> I'm talking about the symlink to clang from clang++, though, which is<br>
> what's busted. I think CMake handles the clang -> clang-3.1 stuff<br>
> internally.<br>
<br>
</div>Ah, sorry.<br>
<br>
Looks like this gets fixed on install already, but it is probably nice<br>
to always use relative symlinks.<br>
<br>
Chandler, any concerns about this patch. Is it OK?</blockquote><div><br></div><div>Sorry for the backlog, conference and committee have delayed my reviews.</div><div><br></div><div>I'm fine in principle. A few issues w/ the patch:</div>
<br>+# Create a relative symlink <br>+ set(clang_binary "./clang${CMAKE_EXECUTABLE_SUFFIX}")</div><div class="gmail_quote"><br></div><div class="gmail_quote">I don't think you want a "./" in the symlink name. Relative symlinks like this usually consist just of the name. Does that not work for any reason? It would make it consistent with the clang -> clang-3.1 symlink.</div>
<div class="gmail_quote"><br></div><div class="gmail_quote">With that fix, as long as it works, LGTM and please commit.</div>