<div dir="ltr">What path should be used in the #include statement?  I'm no CMake expert, but I interpreted add_clang_library only to affect the linking behavior, not what headers are visible.</div><br><div class="gmail_quote"><div dir="ltr">On Mon, May 14, 2018 at 9:31 AM Alexander Kornienko <<a href="mailto:alexfh@google.com">alexfh@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Mon, Apr 23, 2018 at 11:10 AM Ilya Biryukov <<a href="mailto:ibiryukov@google.com" target="_blank">ibiryukov@google.com</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div>You're right, all of clang-extra-tools are not structured as libraries. I guess the most principled approach would to follow LLVM's convention and move public headers into include/, everything else to lib/, etc.</div><div><br></div><div><a class="gmail_plusreply" id="m_6119449432003443972gmail-m_6129691065914596756m_6355262046795646527plusReplyChip-1" href="mailto:alexfh@google.com" target="_blank">+Alexander Kornienko</a>, have you considered extracting a public library interface for clang-tidy before? Any ideas on how to do it properly?</div></div></div></blockquote><div><br></div></div></div><div dir="ltr"><div class="gmail_quote"><div>Clang-tidy has been used as a library since its appearance. But this was done outside the official source tree. If there's interest in using it from clangd, but something is missing in the appropriate CMake configs, we should fix them. However, it looks like tools/extra/clang-tidy/CMakeLists.txt already defines `add_clang_library(clangTidy ...`, which should be enough?</div></div></div>
</blockquote></div>