<div dir="ltr">I'm not familiar with that $<TARGET_FILE> expression, but if you're certain that it's equivalent to what was there previously, this looks fine.<div><br></div><div>But why were the two "Testing LLDB..." messages removed?</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Aug 27, 2015 at 3:55 PM Stephane Sezer <<a href="mailto:sas@cd80.net">sas@cd80.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">sas created this revision.<br>
sas added reviewers: zturner, clayborg.<br>
sas added a subscriber: lldb-commits.<br>
<br>
Instead of hardcoding the path, we should use $<TARGET_FILE:...> which<br>
works better when the output path differs, e.g. when building with MSVC.<br>
<br>
$<TARGET_FILE:...> is expanded during the generation of the final build<br>
files, so we can't use it in output messages/comments. This patch<br>
modifies build messages to display the full command instead of "Testing<br>
LLDB with ..." messages.<br>
<br>
<a href="http://reviews.llvm.org/D12415" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12415</a><br>
<br>
Files:<br>
  test/CMakeLists.txt<br>
<br>
Index: test/CMakeLists.txt<br>
===================================================================<br>
--- test/CMakeLists.txt<br>
+++ test/CMakeLists.txt<br>
@@ -1,13 +1,12 @@<br>
-function(add_python_test_target name test_script args comment)<br>
+function(add_python_test_target name test_script args)<br>
   set(PYTHON_TEST_COMMAND<br>
     ${PYTHON_EXECUTABLE}<br>
     ${test_script}<br>
     ${args}<br>
     )<br>
<br>
   add_custom_target(${name}<br>
     COMMAND ${PYTHON_TEST_COMMAND} ${ARG_DEFAULT_ARGS}<br>
-    COMMENT "${comment}"<br>
     )<br>
 endfunction()<br>
<br>
@@ -35,7 +34,7 @@<br>
 set(LLDB_TEST_COMMON_ARGS<br>
   --arch=${LLDB_TEST_ARCH}<br>
   --executable<br>
-  ${CMAKE_BINARY_DIR}/bin/lldb${CMAKE_EXECUTABLE_SUFFIX}<br>
+  $<TARGET_FILE:lldb><br>
   -s<br>
   ${CMAKE_BINARY_DIR}/lldb-test-traces<br>
   -u CXXFLAGS<br>
@@ -63,7 +62,6 @@<br>
 add_python_test_target(check-lldb-single<br>
   ${LLDB_SOURCE_DIR}/test/dotest.py<br>
   "${LLDB_TEST_COMMON_ARGS};${LLDB_TEST_USER_ARGS}"<br>
-  "Testing LLDB with args: ${LLDB_TEST_COMMON_ARGS};${LLDB_TEST_USER_ARGS}"<br>
   )<br>
<br>
 set(LLDB_DOSEP_ARGS -o;\"-q;${LLDB_TEST_COMMON_ARGS};${LLDB_TEST_USER_ARGS}\")<br>
@@ -73,5 +71,4 @@<br>
 add_python_test_target(check-lldb<br>
   ${LLDB_SOURCE_DIR}/test/dosep.py<br>
   "${LLDB_DOSEP_ARGS}"<br>
-  "Testing LLDB (with a separate subprocess per test)"<br>
   )<br>
<br>
<br>
</blockquote></div>