diff options
author | Alex Neundorf <neundorf@kde.org> | 2010-08-26 19:02:40 (GMT) |
---|---|---|
committer | Alex Neundorf <neundorf@kde.org> | 2010-08-26 19:02:40 (GMT) |
commit | c7866351494885cee1d1d5747ec2bc03fe845f6f (patch) | |
tree | 79a841ea10e378e70a94f26ad037823097c84496 | |
parent | 43f96eaf93541b6e820ca1ebc15802aaa38427e2 (diff) | |
download | CMake-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
-rw-r--r-- | Source/cmTargetLinkLibrariesCommand.cxx | 12 |
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; |