<div dir="auto"><div dir="auto">Oops, right.</div>check-clangd is the way to run clangd tests, it works as before.<div dir="auto">check-clang-tools no longer runs clangd tests.</div><div dir="auto">check-all runs clangd tests if clang-tools-extra is enabled, as before. (Build bots use this method, so should be unaffected)</div><div dir="auto"><br></div><div dir="auto">Also, apologies for butchering the XPC rules, thanks Jan for cleaning up my mess!</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 30, 2019, 04:32 Marc-Andre Laperle <<a href="mailto:malaperle@gmail.com">malaperle@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space">Does this change anything to how the tests need to be executed? i.e. check-clangd, etc?<div>Thanks!</div><div><div><br><blockquote type="cite"><div>On Apr 29, 2019, at 3:23 AM, Sam McCall via clangd-dev <<a href="mailto:clangd-dev@lists.llvm.org" target="_blank" rel="noreferrer">clangd-dev@lists.llvm.org</a>> wrote:</div><br class="m_-5250625782161905842Apple-interchange-newline"><div><div dir="ltr"><div>I'm planning to land a patch that makes the clangd directory structure more self-contained.</div><div><div><br></div><div>Before:</div><div>  clang-tools-extra/clangd</div><div><div></div><div>  clang-tools-extra/test/clangd</div></div><div><div></div><div>  clang-tools-extra/unittests/clangd</div></div><div></div></div><div><div><br></div><div>After:</div><div>  clang-tools-extra/clangd</div><div><div></div><div>  clang-tools-extra/clangd/test</div></div><div><div></div><div>  clang-tools-extra/clangd/unittests</div></div><div></div></div><div><br></div><div>Most people I've talked to agree this is... less annoying.</div><div>But it will temporarily be more annoying, because this will touch every test file and conflict with all your patches. Sorry :-(</div><div><br></div><a href="https://reviews.llvm.org/D61187" target="_blank" rel="noreferrer">https://reviews.llvm.org/D61187</a><br></div>
_______________________________________________<br>clangd-dev mailing list<br><a href="mailto:clangd-dev@lists.llvm.org" target="_blank" rel="noreferrer">clangd-dev@lists.llvm.org</a><br><a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/clangd-dev" target="_blank" rel="noreferrer">https://lists.llvm.org/cgi-bin/mailman/listinfo/clangd-dev</a><br></div></blockquote></div><br></div></div></blockquote></div>