Skip to content

Commit

Permalink
Remove explicit rmm logger link as it is implicit from libcudf (#2634)
Browse files Browse the repository at this point in the history
Signed-off-by: Jason Lowe <[email protected]>
  • Loading branch information
jlowe authored Nov 28, 2024
1 parent 2035c5b commit 6826df4
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 4 deletions.
2 changes: 0 additions & 2 deletions src/main/cpp/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -281,8 +281,6 @@ target_link_libraries(
${ARROW_LIB}
${PARQUET_LIB}
${THRIFT_LIB}
PUBLIC rmm::rmm
PRIVATE $<TARGET_NAME_IF_EXISTS:rmm::rmm_logger_impl>
)
rapids_cuda_set_runtime(spark_rapids_jni USE_STATIC ON)
set_target_properties(spark_rapids_jni PROPERTIES LINK_LANGUAGE "CXX")
Expand Down
2 changes: 1 addition & 1 deletion thirdparty/cudf-pins/versions.json
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@
{
"always_download" : true,
"git_shallow" : false,
"git_tag" : "f9b9f843466b2dcd8872f1707d274ad15be37324",
"git_tag" : "d4066fa611c803430c9bc5dbe8e243f89bb9a25c",
"git_url" : "https://github.com/rapidsai/rmm.git",
"version" : "25.02"
},
Expand Down

0 comments on commit 6826df4

Please sign in to comment.