Merge topic 'clarify-add_dependencies-error'

de13d68 add_dependencies: Distinguish target v. file dependencies in error (#14050)

Source/cmAddDependenciesCommand.cxx

@@ -35,10 +35,14 @@ bool cmAddDependenciesCommand
     }
   else
     {
-    std::string error = "Adding dependency to non-existent target: ";
+    cmOStringStream e;
-    error += target_name;
+    e << "Cannot add target-level dependencies to non-existent target \""
-    this->SetError(error.c_str());
+      << target_name << "\".\n"
-    return false;
+      << "The add_dependencies works for top-level logical targets created "
+      << "by the add_executable, add_library, or add_custom_target commands.  "
+      << "If you want to add file-level dependencies see the DEPENDS option "
+      << "of the add_custom_target and add_custom_command commands.";
+    this->Makefile->IssueMessage(cmake::FATAL_ERROR, e.str());
     }
 
   return true;

Tests/RunCMake/CMakeLists.txt

@@ -68,6 +68,7 @@ if(NOT WIN32)
 endif()
 add_RunCMake_test(CompatibleInterface)
 
+add_RunCMake_test(add_dependencies)
 add_RunCMake_test(build_command)
 add_RunCMake_test(find_package)
 add_RunCMake_test(include)

Tests/RunCMake/add_dependencies/CMakeLists.txt

@@ -0,0 +1,3 @@
+cmake_minimum_required(VERSION 2.8)
+project(${RunCMake_TEST} NONE)
+include(${RunCMake_TEST}.cmake)

Tests/RunCMake/add_dependencies/NoTarget-result.txt

@@ -0,0 +1 @@
+1

Tests/RunCMake/add_dependencies/NoTarget-stderr.txt

@@ -0,0 +1,9 @@
+CMake Error at NoTarget.cmake:1 \(add_dependencies\):
+  Cannot add target-level dependencies to non-existent target "foo".
+
+  The add_dependencies works for top-level logical targets created by the
+  add_executable, add_library, or add_custom_target commands.  If you want to
+  add file-level dependencies see the DEPENDS option of the add_custom_target
+  and add_custom_command commands.
+Call Stack \(most recent call first\):
+  CMakeLists.txt:3 \(include\)

Tests/RunCMake/add_dependencies/NoTarget.cmake

@@ -0,0 +1 @@
+add_dependencies(foo bar)

Tests/RunCMake/add_dependencies/RunCMakeTest.cmake

@@ -0,0 +1,3 @@
+include(RunCMake)
+
+run_cmake(NoTarget)