<div dir="ltr">I didn't actually get around to this yesterday. Landed minutes ago as 99451b4453688a94c6014cac233d371ab4cc342d. I'll keep an eye out for failures, but poke me if I miss something :)</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Feb 10, 2020 at 9:21 AM Jonas Devlieghere <<a href="mailto:jonas@devlieghere.com">jonas@devlieghere.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I'm very excited about this. Thank you for taking the time and effort<br>
to make this happen!<br>
<br>
On Mon, Feb 10, 2020 at 9:01 AM Jordan Rupprecht <<a href="mailto:rupprecht@google.com" target="_blank">rupprecht@google.com</a>> wrote:<br>
><br>
> Later today I'm planning to land D71151, which moves lldb/packages/Python/lldbsuite/test to lldb/test/API, and removes the lldb/test/API/testcases symlink. This is a large move, so I expect it will cause conflict for many outstanding patches with lldb tests. However, it should hopefully make testing the lldbsuite/test/api a little easier: for example, you can now run "ninja check-lldb-api-lang" to run that subdirectory of tests, which matches how ninja targets work for the rest of llvm testing. (Note: "ninja check-lldb" still works without re-invoking cmake, but you need to run cmake again to get all the sub targets). It also removes the symlink, which confuses some tools, and makes file navigation confusing.<br>
><br>
> I have verified no tests got lost in the move via:<br>
> (cd /path/to/src/llvm-build/tools/lldb/test && /usr/bin/python /path/to/src/llvm-build/dev/./bin/llvm-lit -sv /path/to/src/llvm-project/lldb/test/API --show-tests)<br>
> which shows no diffs before/after my patch<br>
</blockquote></div>