summaryrefslogtreecommitdiffstats
path: root/Source/cmTargetLinkLibrariesCommand.cxx
diff options
context:
space:
mode:
authorAlex Neundorf <neundorf@kde.org>2010-08-26 19:02:40 (GMT)
committerAlex Neundorf <neundorf@kde.org>2010-08-26 19:02:40 (GMT)
commitc7866351494885cee1d1d5747ec2bc03fe845f6f (patch)
tree79a841ea10e378e70a94f26ad037823097c84496 /Source/cmTargetLinkLibrariesCommand.cxx
parent43f96eaf93541b6e820ca1ebc15802aaa38427e2 (diff)
downloadCMake-c7866351494885cee1d1d5747ec2bc03fe845f6f.zip
CMake-c7866351494885cee1d1d5747ec2bc03fe845f6f.tar.gz
CMake-c7866351494885cee1d1d5747ec2bc03fe845f6f.tar.bz2
Make target_link_libraries() complain if bad target name is used
target_link_libraries() did not complain if there was only one argument, and this one (first) argument wasn't a valid target name, e.g. add_executable(hello main.cpp) target_link_libraries(-static-intel) Here the target "hello" was forgotten in the tll() call, but cmake didn't complain. With this commit it does. Alex
Diffstat (limited to 'Source/cmTargetLinkLibrariesCommand.cxx')
-rw-r--r--Source/cmTargetLinkLibrariesCommand.cxx12
1 files changed, 6 insertions, 6 deletions
diff --git a/Source/cmTargetLinkLibrariesCommand.cxx b/Source/cmTargetLinkLibrariesCommand.cxx
index d994260..390e39b 100644
--- a/Source/cmTargetLinkLibrariesCommand.cxx
+++ b/Source/cmTargetLinkLibrariesCommand.cxx
@@ -29,12 +29,6 @@ bool cmTargetLinkLibrariesCommand
return false;
}
- // but we might not have any libs after variable expansion
- if(args.size() < 2)
- {
- return true;
- }
-
// Lookup the target for which libraries are specified.
this->Target =
this->Makefile->GetCMakeInstance()
@@ -49,6 +43,12 @@ bool cmTargetLinkLibrariesCommand
return true;
}
+ // but we might not have any libs after variable expansion
+ if(args.size() < 2)
+ {
+ return true;
+ }
+
// Keep track of link configuration specifiers.
cmTarget::LinkLibraryType llt = cmTarget::GENERAL;
bool haveLLT = false;