Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use the new RAPIDS.cmake to fetch rapids-cmake #838

Merged
merged 2 commits into from
Jul 27, 2021
Merged

Use the new RAPIDS.cmake to fetch rapids-cmake #838

merged 2 commits into from
Jul 27, 2021

Commits on Jul 26, 2021

  1. Use the new RAPIDS.cmake to fetch rapids-cmake 

    The original approach of using FetchContent naively has a subtle
bug when multiple projects that use rapids-cmake are combined together inside as sibling projects. This bug causes any
`include(rapids-*)` commands to fail, causing CMake errors.

Bug using `RAPIDS.cmake` we can resolve this issue and remove
the new complex logic from each consumer.
    @robertmaynard
    robertmaynard committed Jul 26, 2021
    Configuration menu
    Copy the full SHA
    a0c1376 View commit details
    Browse the repository at this point in the history

Commits on Jul 27, 2021

  1. Correct CMake style issues found by CI

    @robertmaynard
    robertmaynard committed Jul 27, 2021
    Configuration menu
    Copy the full SHA
    5950041 View commit details
    Browse the repository at this point in the history