Skip to content

Commit 68d1368

Browse files
authored
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 <raulcumplido@gmail.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
1 parent a444c87 commit 68d1368

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

cpp/src/arrow/integration/CMakeLists.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,9 @@ elseif(ARROW_BUILD_INTEGRATION)
3333

3434
add_dependencies(arrow-json-integration-test arrow arrow_testing)
3535
add_dependencies(arrow-integration arrow-json-integration-test)
36+
endif()
3637

38+
if(ARROW_BUILD_INTEGRATION)
3739
add_arrow_lib(arrow_c_data_integration
3840
SOURCES
3941
c_data_integration_internal.cc

0 commit comments

Comments
 (0)