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 68d13685b4 GH-48862: [C++][Integration] Build arrow_c_data_integration 
library regardless of ARROW_TEST value (#49236)
68d13685b4 is described below

commit 68d13685b4e3914ff2b36eeca606ae428b8e3485
Author: Raúl Cumplido <[email protected]>
AuthorDate: Thu Feb 12 06:35:45 2026 +0100

    GH-48862: [C++][Integration] Build arrow_c_data_integration library 
regardless of ARROW_TEST value (#49236)
    
    ### Rationale for this change
    
    Currently if `ARROW_TEST=ON` the arrow_c_data_integration library is not 
built. This has been seen on Release verification for 23.0.0 and 23.0.1.
    
    ### What changes are included in this PR?
    
    Build `arrow_c_data_integration` library if `ARROW_BUILD_INTEGRATION=ON` 
regardless of `ARROW_TEST` value.
    
    ### Are these changes tested?
    
    Yes, I've temporarily modified the crossbow job to run both tests and 
integration and validated the commit fixes the problem and it failed without 
the fix.
    
    ### Are there any user-facing changes?
    
    No
    
    * GitHub Issue: #48862
    
    Authored-by: Raúl Cumplido <[email protected]>
    Signed-off-by: Sutou Kouhei <[email protected]>
---
 cpp/src/arrow/integration/CMakeLists.txt | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/cpp/src/arrow/integration/CMakeLists.txt 
b/cpp/src/arrow/integration/CMakeLists.txt
index fd239ff2ab..267d0adf11 100644
--- a/cpp/src/arrow/integration/CMakeLists.txt
+++ b/cpp/src/arrow/integration/CMakeLists.txt
@@ -33,7 +33,9 @@ elseif(ARROW_BUILD_INTEGRATION)
 
   add_dependencies(arrow-json-integration-test arrow arrow_testing)
   add_dependencies(arrow-integration arrow-json-integration-test)
+endif()
 
+if(ARROW_BUILD_INTEGRATION)
   add_arrow_lib(arrow_c_data_integration
                 SOURCES
                 c_data_integration_internal.cc

Reply via email to