rapidsai / rapids-cmake

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

[DOC] Expand documentation around rapids-cmake cpm overrides in `rapids_cpm_versions.rst` #551

Closed robertmaynard closed 3 months ago

robertmaynard commented 6 months ago

Report incorrect documentation

The documentation around the usage of rapids-cmake cpm overrides is missing some of the edge cases:

  1. always_download will be true for any package that is inside an override, due to the call to FetchContent_Declare
  2. The docs around how multiple overrides interact should be in rapids_cpm_versions.rst as well as package_override.rst
  3. Document CPM_<package_name>_SOURCE behavior in regards to all rapids_cpm calls
  4. Document that the existence of a patch entry in either the base package definition or the override will cause no find_package call to occur.
robertmaynard commented 3 months ago

613 resolves this issue