diff options
| author | Brad King <brad.king@kitware.com> | 2017-09-26 11:14:38 (GMT) |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2017-09-26 11:31:33 (GMT) |
| commit | 1a7b8c83210c2574652c5045e3338a7ccba03734 (patch) | |
| tree | 2b375bfe6bce75d12b121797bb46c7d9e76eaa4e /Modules/GetPrerequisites.cmake | |
| parent | eae3765b67b653d3f00afa44a60719a387262af8 (diff) | |
| download | CMake-1a7b8c83210c2574652c5045e3338a7ccba03734.zip CMake-1a7b8c83210c2574652c5045e3338a7ccba03734.tar.gz CMake-1a7b8c83210c2574652c5045e3338a7ccba03734.tar.bz2 | |
GetPrerequisites: Restore behavior on missing binary of not clearing list
Prior to commit v3.4.0-rc1~264^2~1 (GetPrerequisites: Add error checks
for execute_process() calls, 2015-07-29), `get_prerequisites` would
simply warn on a missing binary and not update the result list at all.
That commit accidentally made the case an error. This was fixed by
commit v3.8.0-rc1~110^2 (GetPrerequisites: Do not fail on files we
cannot find, 2017-01-10), but the fix also cleared the result list.
Clearing the list is incorrect because it is supposed to be able to
accumulate results over multiple calls.
Remove the list clearing behavior to restore the original behavior on a
missing binary.
Fixes: #17306
Diffstat (limited to 'Modules/GetPrerequisites.cmake')
| -rw-r--r-- | Modules/GetPrerequisites.cmake | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/Modules/GetPrerequisites.cmake b/Modules/GetPrerequisites.cmake index 4e52cb3..b81dd76 100644 --- a/Modules/GetPrerequisites.cmake +++ b/Modules/GetPrerequisites.cmake @@ -659,7 +659,6 @@ function(get_prerequisites target prerequisites_var exclude_system recurse exepa if(NOT EXISTS "${target}") message("warning: target '${target}' does not exist...") - set(${prerequisites_var} "" PARENT_SCOPE) return() endif() |