diff options
| author | scivision <scivision@users.noreply.github.com> | 2023-01-23 15:47:54 (GMT) |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2023-01-24 19:13:52 (GMT) |
| commit | a1dc38a5670939a8ce8dfd99282f86d47412a98a (patch) | |
| tree | 12f0296080cf6c345901839bc6f134fbd01177d3 /Tests/RunCMake/if/IsDirectoryLong.cmake | |
| parent | adac7f3ca41ae38248f62ce48000390143762e99 (diff) | |
| download | CMake-a1dc38a5670939a8ce8dfd99282f86d47412a98a.zip CMake-a1dc38a5670939a8ce8dfd99282f86d47412a98a.tar.gz CMake-a1dc38a5670939a8ce8dfd99282f86d47412a98a.tar.bz2 | |
Help: Document behavior of if(EXISTS,IS_DIRECTORY) for empty paths
Document and add explicit tests for empty string input:
`if(EXISTS "")` and `if(IS_DIRECTORY "")` are always false.
This avoids need for users to do extra checks due to
CMake non-short-circuit logic as below:
if("${p}")
if(EXISTS "${p}")
Diffstat (limited to 'Tests/RunCMake/if/IsDirectoryLong.cmake')
| -rw-r--r-- | Tests/RunCMake/if/IsDirectoryLong.cmake | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/Tests/RunCMake/if/IsDirectoryLong.cmake b/Tests/RunCMake/if/IsDirectoryLong.cmake index e6939ac..41f1818 100644 --- a/Tests/RunCMake/if/IsDirectoryLong.cmake +++ b/Tests/RunCMake/if/IsDirectoryLong.cmake @@ -8,3 +8,6 @@ if(IS_DIRECTORY "${d}/") else() message(STATUS "Directory path with length ${dl} correctly does not exist.") endif() +if(IS_DIRECTORY "") + message(FATAL_ERROR "IS_DIRECTORY \"\" should not exist") +endif() |