https://github.com/Kitware/CMake
Revision d9a6e0ffc82fbc460289c6b0808eff27ac316235 authored by Craig Scott on 10 September 2022, 08:22:18 UTC, committed by Craig Scott on 10 September 2022, 08:23:21 UTC
The unset() command was using __cmake_contentNameLower before that variable was restored from the __cmake_fcCurrentVarsStack. That means if there had been a nested call to FetchContent_MakeAvailable(), the wrong variable name would have been cleared (the nested name instead of the one from the current call). That would have left the variable set upon return, blocking the dependency provider from seeing any further calls to FetchContent_MakeAvailable() in the current variable scope or below for the current dependency.
1 parent a2f9e67
Tip revision: d9a6e0ffc82fbc460289c6b0808eff27ac316235 authored by Craig Scott on 10 September 2022, 08:22:18 UTC
FetchContent: Fix unsetting wrong variable name after provider returns
FetchContent: Fix unsetting wrong variable name after provider returns
Tip revision: d9a6e0f
| File | Mode | Size |
|---|---|---|
| GeneratedHeader | ||
| GeneratorInExtraDir | ||
| CMakeLists.txt | -rw-r--r-- | 18.6 KB |
| check_command_line.c.in | -rw-r--r-- | 687 bytes |
| check_mark.cmake | -rw-r--r-- | 173 bytes |
| compare_options.cmake | -rw-r--r-- | 372 bytes |
| doc1.tex | -rw-r--r-- | 23 bytes |
| empty_command.cxx | -rw-r--r-- | 27 bytes |
| foo.h.in | -rw-r--r-- | 61 bytes |
| foo.in | -rw-r--r-- | 412 bytes |
| gen_redirect_in.c | -rw-r--r-- | 96 bytes |
| generator.cxx | -rw-r--r-- | 406 bytes |
| mac_fw.c | -rw-r--r-- | 33 bytes |
| main.cxx | -rw-r--r-- | 62 bytes |
| source_in_custom_target.cpp | -rw-r--r-- | 0 bytes |
| subdir.h.in | -rw-r--r-- | 19 bytes |
| tcat.cxx | -rw-r--r-- | 121 bytes |
| wrapped.h | -rw-r--r-- | 17 bytes |
| wrapper.cxx | -rw-r--r-- | 752 bytes |
Computing file changes ...
