diff options
author | Brad King <brad.king@kitware.com> | 2012-07-10 10:55:02 -0400 |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2012-07-10 11:05:04 -0400 |
commit | 2e99949ccb216c59ff1932bc3773654e9ae54419 (patch) | |
tree | e0a7ee0de24eab9b10f1fcfceead009d0faf2f3b /Source/cmIncludeCommand.cxx | |
parent | 6274ca6f910683495124d22e371a1adc9cf0eebc (diff) | |
download | cmake-2e99949ccb216c59ff1932bc3773654e9ae54419.tar.gz |
include: Ignore empty string as file name (#13388)
Previously CMake silently accepted the empty string and added a bogus
dependency on the current directory. Instead warn about the empty file
name and ignore it. We cannot make this an error because there may be
existing projects that accidentally depend on the old behavior.
Add a RunCMake.include test to cover this case.
Diffstat (limited to 'Source/cmIncludeCommand.cxx')
-rw-r--r-- | Source/cmIncludeCommand.cxx | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/Source/cmIncludeCommand.cxx b/Source/cmIncludeCommand.cxx index 0ac6df46ac..0d5f67b78e 100644 --- a/Source/cmIncludeCommand.cxx +++ b/Source/cmIncludeCommand.cxx @@ -70,6 +70,13 @@ bool cmIncludeCommand } } + if(fname.empty()) + { + this->Makefile->IssueMessage(cmake::AUTHOR_WARNING, + "include() given empty file name (ignored)."); + return true; + } + if(!cmSystemTools::FileIsFullPath(fname.c_str())) { // Not a path. Maybe module. |