From 08aefc395d5763e405c736c35231706f6f9634a2 Mon Sep 17 00:00:00 2001 From: Gang Wu Date: Mon, 22 Apr 2024 20:52:57 +0800 Subject: [PATCH] MINOR: [C++][CMake] Remove unused CMAKE_SKIP_INSTALL_ALL_DEPENDENCY (#41332) ### Rationale for this change CMAKE_SKIP_INSTALL_ALL_DEPENDENCY was removed in https://github.com/apache/arrow/pull/75 but it seems that there is still one line remaining. ### What changes are included in this PR? Remove unused CMAKE_SKIP_INSTALL_ALL_DEPENDENCY. ### Are these changes tested? Pass all CIs. ### Are there any user-facing changes? No. Authored-by: Gang Wu Signed-off-by: Gang Wu --- cpp/CMakeLists.txt | 3 --- 1 file changed, 3 deletions(-) diff --git a/cpp/CMakeLists.txt b/cpp/CMakeLists.txt index 1499bb7767d4a..cc4df6d2fa2eb 100644 --- a/cpp/CMakeLists.txt +++ b/cpp/CMakeLists.txt @@ -256,9 +256,6 @@ if(ARROW_USE_CCACHE endif() if(ARROW_OPTIONAL_INSTALL) - # Don't make the "install" target depend on the "all" target - set(CMAKE_SKIP_INSTALL_ALL_DEPENDENCY true) - set(INSTALL_IS_OPTIONAL OPTIONAL) endif()