MINOR: [C++][CMake] Remove unused CMAKE_SKIP_INSTALL_ALL_DEPENDENCY (a…

…pache#41332) ### Rationale for this change CMAKE_SKIP_INSTALL_ALL_DEPENDENCY was removed in apache#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 <[email protected]> Signed-off-by: Gang Wu <[email protected]>
CurtHagenlocher · Apr 22, 2024 · 08aefc3 · 08aefc3
1 parent 25bb627
commit 08aefc3
Showing 1 changed file with 0 additions and 3 deletions.
diff --git 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()