From 6aef6d84d7faa5e5f0e93237f991a8ef134ddff9 Mon Sep 17 00:00:00 2001 From: Alex Neundorf Date: Thu, 26 Aug 2010 21:54:43 +0200 Subject: Just warn in case of a bad target as only argument for t_l_l() As discussed on cmake-devel, if target_link_libraries() is called with only one argument, and this one argument is not a valid target, just print a warning but don't fail, since otherwise probably some existing code somewhere might stop building. Alex --- Source/cmTargetLinkLibrariesCommand.cxx | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/Source/cmTargetLinkLibrariesCommand.cxx b/Source/cmTargetLinkLibrariesCommand.cxx index 390e39b..e92a8fa 100644 --- a/Source/cmTargetLinkLibrariesCommand.cxx +++ b/Source/cmTargetLinkLibrariesCommand.cxx @@ -38,8 +38,18 @@ bool cmTargetLinkLibrariesCommand cmOStringStream e; e << "Cannot specify link libraries for target \"" << args[0] << "\" " << "which is not built by this project."; - this->Makefile->IssueMessage(cmake::FATAL_ERROR, e.str()); - cmSystemTools::SetFatalErrorOccured(); + // The bad target is the only argument, just warn, don't fail, because + // there is probably some code out there which would stop building + // otherwise: + if (args.size() < 2) + { + this->Makefile->IssueMessage(cmake::AUTHOR_WARNING, e.str()); + } + else + { + this->Makefile->IssueMessage(cmake::FATAL_ERROR, e.str()); + cmSystemTools::SetFatalErrorOccured(); + } return true; } -- cgit v0.12