This is an automated email from the ASF dual-hosted git repository.

kou pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/main by this push:
     new a0f9d2eb2f GH-41617: [C++][CMake] Fix ARROW_USE_BOOST detect condition 
(#41622)
a0f9d2eb2f is described below

commit a0f9d2eb2fe6743a869f3509f456389cf3de4926
Author: Sutou Kouhei <[email protected]>
AuthorDate: Sun May 12 20:09:55 2024 +0900

    GH-41617: [C++][CMake] Fix ARROW_USE_BOOST detect condition (#41622)
    
    ### Rationale for this change
    
    We also need Boost when ARROW_FLIGHT and ARROW_TESTING are true.
    
    ### What changes are included in this PR?
    
    Add missing the condition.
    
    ### Are these changes tested?
    
    Yes.
    
    ### Are there any user-facing changes?
    
    Yes.
    * GitHub Issue: #41617
    
    Authored-by: Sutou Kouhei <[email protected]>
    Signed-off-by: Sutou Kouhei <[email protected]>
---
 cpp/cmake_modules/ThirdpartyToolchain.cmake | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cpp/cmake_modules/ThirdpartyToolchain.cmake 
b/cpp/cmake_modules/ThirdpartyToolchain.cmake
index ddea1c399c..2102a7fdcd 100644
--- a/cpp/cmake_modules/ThirdpartyToolchain.cmake
+++ b/cpp/cmake_modules/ThirdpartyToolchain.cmake
@@ -1263,7 +1263,7 @@ endif()
 # - S3FS and Flight benchmarks need Boost at runtime.
 if(ARROW_BUILD_INTEGRATION
    OR ARROW_BUILD_TESTS
-   OR (ARROW_FLIGHT AND ARROW_BUILD_BENCHMARKS)
+   OR (ARROW_FLIGHT AND (ARROW_TESTING OR ARROW_BUILD_BENCHMARKS))
    OR (ARROW_S3 AND ARROW_BUILD_BENCHMARKS))
   set(ARROW_USE_BOOST TRUE)
   set(ARROW_BOOST_REQUIRE_LIBRARY TRUE)

Reply via email to