Skip to content

Commit

Permalink
Remove zlib from public CMake dependencies
Browse files Browse the repository at this point in the history
  • Loading branch information
robertmaynard committed Sep 9, 2021
1 parent 473063f commit 7da6c3a
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions cpp/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -102,9 +102,8 @@ include(cmake/Modules/ConfigureCUDA.cmake) # set other CUDA compilation flags
# - dependencies ----------------------------------------------------------------------------------

# find zlib
rapids_find_package(ZLIB REQUIRED
BUILD_EXPORT_SET cudf-exports
INSTALL_EXPORT_SET cudf-exports)
rapids_find_package(ZLIB REQUIRED)

# find Threads (needed by cudftestutil)
rapids_find_package(Threads REQUIRED
BUILD_EXPORT_SET cudf-exports
Expand Down Expand Up @@ -505,11 +504,11 @@ add_dependencies(cudf jitify_preprocess_run)

# Specify the target module library dependencies
target_link_libraries(cudf
PUBLIC ZLIB::ZLIB
${ARROW_LIBRARIES}
PUBLIC ${ARROW_LIBRARIES}
cudf::Thrust
rmm::rmm
PRIVATE cuco::cuco)
PRIVATE cuco::cuco
ZLIB::ZLIB)

# Add Conda library, and include paths if specified
if(TARGET conda_env)
Expand Down

0 comments on commit 7da6c3a

Please sign in to comment.