diff options
| author | Craig Scott <craig.scott@crascit.com> | 2024-05-03 23:49:58 (GMT) |
|---|---|---|
| committer | Craig Scott <craig.scott@crascit.com> | 2024-05-12 04:45:43 (GMT) |
| commit | 91e1015722e2e62249839ccb0f557b810740e6fb (patch) | |
| tree | f5ce8e570929b963a377ff83760499524cd4b34f /Modules | |
| parent | a1743ce1efe57526433623c45251aeeef63413f8 (diff) | |
| download | CMake-91e1015722e2e62249839ccb0f557b810740e6fb.zip CMake-91e1015722e2e62249839ccb0f557b810740e6fb.tar.gz CMake-91e1015722e2e62249839ccb0f557b810740e6fb.tar.bz2 | |
ExternalProject: Don't treat YES as a keyword
This ultimately won't change behavior, since YES should never be listed
in the "keywords" argument. The original code looks like a bug though,
since YES satisfies the criteria for being interpreted as a keyword, and
there is logic to short-circuit on TRUE and FALSE. Add YES to that
condition to avoid any potential confusion.
Diffstat (limited to 'Modules')
| -rw-r--r-- | Modules/ExternalProject.cmake | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Modules/ExternalProject.cmake b/Modules/ExternalProject.cmake index 02baabe..7ea6387 100644 --- a/Modules/ExternalProject.cmake +++ b/Modules/ExternalProject.cmake @@ -1341,7 +1341,7 @@ function(_ep_parse_arguments if(arg MATCHES "^[A-Z][A-Z0-9_][A-Z0-9_]+$" AND NOT (("x${arg}x" STREQUAL "x${key}x") AND ("x${key}x" STREQUAL "xCOMMANDx")) AND - NOT arg MATCHES "^(TRUE|FALSE)$") + NOT arg MATCHES "^(TRUE|FALSE|YES)$") if(arg IN_LIST keywords) set(is_value 0) endif() |