rapidsai / rapids-cmake

https://docs.rapids.ai/api/rapids-cmake/stable/
Apache License 2.0
27 stars 44 forks source link

[FEA] rapids-cmake `GLOBAL_TARGETS ....` should just infer `CMAKE_FIND_PACKAGE_TARGETS_GLOBAL=ON` #626

Open robertmaynard opened 1 month ago

robertmaynard commented 1 month ago

Is your feature request related to a problem? Please describe. Currently developers need to explicitly list the targets they want evalated to global state. Now that we require CMake 3.24 we should remove that burden of listing targets from users of rapids-cmake.

Describe the solution you'd like

Every rapids-cmake API that has a GLOBAL_TARGETS parameter should support zero values. When we are given GLOBAL_TARGETS we should enable CMAKE_FIND_PACKAGE_TARGETS_GLOBAL to make all targets global and not just the ones listed.