Set CUDA_RUNTIME_LIBRARY to documented case style

[KyleFromNVIDIA]

CMake documentation lists "Shared" and "Static", not "SHARED" and "STATIC". Set the value with the documented casing.

Description

Checklist

• I am familiar with the Contributing Guidelines.
• New or existing tests cover these changes.
• The documentation is up to date with these changes.
• The cmake-format.json is up to date with these changes.
• I have added new files under rapids-cmake/
  • I have added include guards (include_guard(GLOBAL))
  • I have added the associated docs/ rst file and update the api.rst

[robertmaynard]

This isn't a breaking change. CMake internally uppercases the value before evaluation so it can support shared, SHARED etc.

https://gitlab.kitware.com/cmake/cmake/-/blob/master/Source/cmGeneratorTarget.cxx?ref_type=heads#L3557

[KyleFromNVIDIA]

Indeed - I had not noticed that when reading the CMake code. Thanks.

[KyleFromNVIDIA]

Given that this PR doesn't really fix anything, I'm fine with just closing it and leaving things as they are.

[robertmaynard]

Given that this PR doesn't really fix anything, I'm fine with just closing it and leaving things as they are.

I think matching the documented public API for this function is a good call. We should towards the public API and not the internal implementation.

I would just like to revise the commit message and title to better reflect that. e.g. "Set CUDA_RUNTIME_LIBRARY to documented case style"

[KyleFromNVIDIA]

OK, I've updated the PR title and description.

[KyleFromNVIDIA]

/merge