LGTM too btw. =] Thanks for doing this.<div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jul 31, 2012 at 12:55 PM, Douglas Gregor <span dir="ltr"><<a href="mailto:dgregor@apple.com" target="_blank">dgregor@apple.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="HOEnZb"><div class="h5"><br>
On Jul 31, 2012, at 12:42 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
<br>
> On Tue, Jul 31, 2012 at 11:27 AM, Douglas Gregor <<a href="mailto:dgregor@apple.com">dgregor@apple.com</a>> wrote:<br>
>><br>
>> On Jul 31, 2012, at 11:20 AM, Manuel Klimek <<a href="mailto:klimek@google.com">klimek@google.com</a>> wrote:<br>
>><br>
>>> On Tue, Jul 31, 2012 at 8:15 PM, Douglas Gregor <<a href="mailto:dgregor@apple.com">dgregor@apple.com</a>> wrote:<br>
>>>><br>
>>>> On Jul 26, 2012, at 10:23 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
>>>><br>
>>>>> While reading the "How To Setup Clang Tooling for LLVM" documentation<br>
>>>>> ( <a href="http://clang.llvm.org/docs/HowToSetupToolingForLLVM.html" target="_blank">http://clang.llvm.org/docs/HowToSetupToolingForLLVM.html</a> ) I ran<br>
>>>>> into a snag where the document implied that clang-check would be<br>
>>>>> installed alongside clang. This is currently not the case - we don't<br>
>>>>> install clang-check, at least not in the cmake build (&, given the<br>
>>>>> presence of "NO_INSTALL = 1" in the Makefile, I assume we don't in the<br>
>>>>> make build either).<br>
>>>>><br>
>>>>> Should we? It seems like a natural enough thing to install, though I<br>
>>>>> realize the specifics of which tools will be developed where and how<br>
>>>>> they'll be installed is still in flux, so I figured I'd start a thread<br>
>>>>> to discuss this rather than just committing it.<br>
>>>>><br>
>>>>> [as a side note: why do we install diagtool (perhaps there's some use<br>
>>>>> for it other than the internal diagnostic flag regression testing?)<br>
>>>>> and c-index-test (by name I would've thought that was just an internal<br>
>>>>> test binary)]<br>
>>>>> <clang_check_install.diff><br>
>>>><br>
>>>><br>
>>>> It depends on whether we think the installation is for end-users of Clang or for developers who want to work on Clang or Clang-based tools. I tend to think that we should favor for former, and only install the base compiler (clang, clang++, support headers and support libraries). If we want to have a "developer mode" that installs everything else, that's fine.<br>

>>><br>
>>> Especially with the vim integration, the use case of clang-check I see<br>
>>> is much more for clang-users (-> compiling their random open source<br>
>>> project with clang) than for clang devs. Of course we're not yet at<br>
>>> the integration level we want to be at for editors; which I can see as<br>
>>> an objection to default-installing it in its current state.<br>
>><br>
>><br>
>> That's a good point; clang-check is (will be) important for users.<br>
>><br>
>> diagtool, though, isn't something users should ever need. If a user needs to explore warnings and flags, they should be able to look at some documentation. Installing diagtool doesn't get us out of writing documentation :)<br>

><br>
> So - stop installing diagtool and c-index-test and start installing clang-check?<br>
<br>
</div></div>WFM.<br>
<br>
        - Doug<br>
<div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>