Speed-up rust build

rust/cuvs-sys/CMakeLists.txt

@@ -0,0 +1,58 @@
+# =============================================================================
+# Copyright (c) 2024, NVIDIA CORPORATION.
+#
+# Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except
+# in compliance with the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software distributed under the License
+# is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express
+# or implied. See the License for the specific language governing permissions and limitations under
+# the License.
+# =============================================================================
+
+cmake_minimum_required(VERSION 3.26.4 FATAL_ERROR)
+
+include(../../rapids_config.cmake)
+
+# we want to use the already built libcuvs if its available, but the rust cmake-rs project doesn't
+# support anything like find_package https://github.com/rust-lang/cmake-rs/issues/111 instead we're
+# adding an extra level of indirection here - cmake-rs will attempt to build this project, and we'll
+# using the existing libcuvs if its already built, and only fall back to building libcuvs if it
+# isn't
+
+project(
+  cuvs-rs
+  VERSION "${RAPIDS_VERSION}"
+  LANGUAGES CXX CUDA
+)
+
+option(FIND_CUVS_CPP "Search for existing CUVS C++ installations before defaulting to local files"
+       ON
+)
+
+# If the user requested it we attempt to find CUVS.
+if(FIND_CUVS_CPP)
+  find_package(cuvs "${RAPIDS_VERSION}" REQUIRED COMPONENTS compiled)
+  if(NOT TARGET cuvs::cuvs)
+    message(
+      FATAL_ERROR
+        "Building against a preexisting libcuvs library requires the compiled libcuvs to have been built!"
+    )
+
+  endif()
+else()
+  set(cuvs_FOUND OFF)
+endif()
+
+if(NOT cuvs_FOUND)
+  set(BUILD_TESTS OFF)
+  set(BUILD_C_LIBRARY ON)
+  add_subdirectory(../../cpp cuvs-cpp EXCLUDE_FROM_ALL)
+endif()
+
+# add a dummy target here,
+add_library(cuvs-rust INTERFACE)
+target_link_libraries(cuvs-rust INTERFACE cuvs::cuvs)
+install(TARGETS cuvs-rust)

rust/cuvs-sys/build.rs

@@ -18,20 +18,9 @@ use std::env;
 use std::io::BufRead;
 use std::path::PathBuf;
 
-/*
-    TODO:
-    * would be nice to use already built versions of libcuvs_c / libcuvs
-        if they already existed, but this might not be possible here using cmake-rs
-        (https://github.com/rust-lang/cmake-rs/issues/111)
-    * figure out how this works with rust packaging: does the c++ code
-        need to be in a subdirectory? If so would a symlink work here
-        should we be using static linking ?
-*/
 fn main() {
     // build the cuvs c-api library with cmake, and link it into this crate
-    let cuvs_build = cmake::Config::new("../../cpp")
-        .configure_arg("-DBUILD_TESTS:BOOL=OFF")
-        .configure_arg("-DBUILD_C_LIBRARY:BOOL=ON")
+    let cuvs_build = cmake::Config::new(".")
         .build();
 
     println!(