diff options
| author | Brad King <brad.king@kitware.com> | 2024-03-11 15:57:41 (GMT) |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2024-03-11 16:07:32 (GMT) |
| commit | b1c48069149fcd9ff128286cb059138b1df96c8f (patch) | |
| tree | de7e7ebc622f49d7b7fe6ecb2be7992cfdf1adb6 /Source | |
| parent | 04d8bc6bb9727b667b8a0d339c7803befce337f2 (diff) | |
| download | CMake-b1c48069149fcd9ff128286cb059138b1df96c8f.zip CMake-b1c48069149fcd9ff128286cb059138b1df96c8f.tar.gz CMake-b1c48069149fcd9ff128286cb059138b1df96c8f.tar.bz2 | |
install(EXPORT): Drop REQUIRED from exported find_dependency calls
In commit c6e6861e63 (install(EXPORT): Export find_dependency() calls,
2023-11-07, v3.29.0-rc1~439^2~1) we made the calls `REQUIRED`. However,
a dependency is only required if the dependent package is required.
`find_dependency` already forwards the `REQUIRED` mark, and also already
marks the dependent package as not found if the dependency is missing.
Fixes: #25756
Diffstat (limited to 'Source')
| -rw-r--r-- | Source/cmExportFileGenerator.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Source/cmExportFileGenerator.cxx b/Source/cmExportFileGenerator.cxx index 37af5f0..f60a17e 100644 --- a/Source/cmExportFileGenerator.cxx +++ b/Source/cmExportFileGenerator.cxx @@ -1260,7 +1260,7 @@ void cmExportFileGenerator::GenerateFindDependencyCalls(std::ostream& os) for (auto const& it : packageDependenciesSorted) { if (it.second.Enabled == cmExportSet::PackageDependencyExportEnabled::On) { - os << "find_dependency(" << it.first << " REQUIRED"; + os << "find_dependency(" << it.first; for (auto const& arg : it.second.ExtraArguments) { os << " " << cmOutputConverter::EscapeForCMake(arg); } |