From 8a644afc77ebe6333114e503cab29f9b0969618a Mon Sep 17 00:00:00 2001 From: Sutou Kouhei Date: Sat, 9 Dec 2023 07:01:31 +0900 Subject: [PATCH] GH-39136: [C++] Remove needless system Protobuf dependency with -DARROW_HDFS=ON (#39137) ### Rationale for this change Our HDFS related codes don't depend on Protobuf because we process HDFS via external `libhdfs.so` and it's `dlopen()`-ed. ### What changes are included in this PR? Remove a needless CMake configuration. ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * Closes: #39136 Authored-by: Sutou Kouhei Signed-off-by: Sutou Kouhei --- cpp/CMakeLists.txt | 3 --- 1 file changed, 3 deletions(-) diff --git a/cpp/CMakeLists.txt b/cpp/CMakeLists.txt index bcb298407bd8b..9f17350b2505a 100644 --- a/cpp/CMakeLists.txt +++ b/cpp/CMakeLists.txt @@ -873,9 +873,6 @@ add_dependencies(arrow_test_dependencies toolchain-tests) if(ARROW_STATIC_LINK_LIBS) add_dependencies(arrow_dependencies ${ARROW_STATIC_LINK_LIBS}) if(ARROW_HDFS OR ARROW_ORC) - if(Protobuf_SOURCE STREQUAL "SYSTEM") - list(APPEND ARROW_STATIC_INSTALL_INTERFACE_LIBS ${ARROW_PROTOBUF_LIBPROTOBUF}) - endif() if(NOT MSVC_TOOLCHAIN) list(APPEND ARROW_STATIC_LINK_LIBS ${CMAKE_DL_LIBS}) list(APPEND ARROW_STATIC_INSTALL_INTERFACE_LIBS ${CMAKE_DL_LIBS})