Install headers into `${CMAKE_INSTALL_INCLUDEDIR}`

rapidsai / rmm

RAPIDS Memory Manager

https://docs.rapids.ai/api/rmm/stable/

Apache License 2.0

478 stars 195 forks source link

Closed KyleFromNVIDIA closed 2 months ago

KyleFromNVIDIA commented 2 months ago

Instead of installing into a hard-coded include directory, install to ${CMAKE_INSTALL_INCLUDEDIR}, which defaults to include.

KyleFromNVIDIA commented 2 months ago

/merge

jakirkham commented 2 months ago

Thanks all! 🙏

harrism commented 2 months ago

All PRs in RMM should have an associated Issue, please. I added this requirement to the PR template in 24.08.