<div dir="ltr">One more nitpick.  Can you make it a dependency of `check-lldb-lit` target also?  Just for the sake of pedantry.</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 27, 2017 at 2:04 PM Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ship it.<br>
<br>
On 27 October 2017 at 13:56, Davide Italiano <<a href="mailto:dccitaliano@gmail.com" target="_blank">dccitaliano@gmail.com</a>> wrote:<br>
> Take 2 (it can't be in the top-level CMakeList because the check-lldb<br>
> target is declared elsewhere).<br>
><br>
> $ git diff HEAD<br>
> diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt<br>
> index d5d71d1..958f9f3 100644<br>
> --- a/test/CMakeLists.txt<br>
> +++ b/test/CMakeLists.txt<br>
> @@ -109,6 +109,12 @@ add_python_test_target(check-lldb<br>
>    "Testing LLDB (parallel execution, with a separate subprocess per test)"<br>
>    )<br>
><br>
> +# If we're building with an in-tree clang, then list clang as a dependency<br>
> +# to run tests.<br>
> +if (TARGET clang)<br>
> +  add_dependencies(check-lldb clang)<br>
> +endif()<br>
> +<br>
>  add_custom_target(lldb-test-depends DEPENDS ${LLDB_TEST_DEPENDS})<br>
>  # This will add LLDB's test dependencies to the depenednecies for check-all and<br>
>  # include them in the test-depends target.<br>
><br>
> On Fri, Oct 27, 2017 at 1:48 PM, Pavel Labath <<a href="mailto:labath@google.com" target="_blank">labath@google.com</a>> wrote:<br>
>> On 27 October 2017 at 13:28, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
>>> Maybe make it a dependency of the `check-lldb` target instead of the `lldb`<br>
>>> target?<br>
>>><br>
>><br>
>> +1<br>
</blockquote></div>