diff --git a/conda/recipes/rmm/meta.yaml b/conda/recipes/rmm/meta.yaml index a1b97e0ed..be4518b32 100644 --- a/conda/recipes/rmm/meta.yaml +++ b/conda/recipes/rmm/meta.yaml @@ -36,8 +36,6 @@ build: requirements: build: - # TODO: See discussion in https://github.com/conda-forge/scikit-build-feedstock/issues/69, - # should cmake/ninja be in this section or in host? - cmake >=3.23.1,!=3.25.0 - ninja - {{ compiler('c') }} diff --git a/python/CMakeLists.txt b/python/CMakeLists.txt index 6baa9d308..b1f166954 100644 --- a/python/CMakeLists.txt +++ b/python/CMakeLists.txt @@ -25,12 +25,7 @@ include(${CMAKE_BINARY_DIR}/RAPIDS.cmake) project( rmm-python VERSION ${rmm_version} - # TODO: See if we can remove the C language when switching to scikit-build-core. - LANGUAGES # TODO: Building Python extension modules via the python_extension_module requires the C - # language to be enabled here. The test project that is built in scikit-build to verify - # various linking options for the python library is hardcoded to build with C, so until - # that is fixed we need to keep C. - C CXX) + LANGUAGES CXX) option(FIND_RMM_CPP "Search for existing RMM C++ installations before defaulting to local files" OFF)