GH-48862: [C++][Integration] Build arrow_c_data_integration library regardless of ARROW_TEST value#49236
Conversation
…rary regardless of ARROW_TEST value
|
@github-actions crossbow submit verify-rc-source-integration-* |
|
|
github-actions
Bot
added
Component: C++
awaiting committer review
|
Revision: df21935 Submitted crossbow builds: ursacomputing/crossbow @ actions-9fefa32cb1
|
|
@github-actions crossbow submit verify-rc-source-integration-linux-ubuntu-24.04-amd64 |
|
Revision: 9cc4e27 Submitted crossbow builds: ursacomputing/crossbow @ actions-83d0a25c50
|
|
@github-actions crossbow submit verify-rc-source-integration-linux-ubuntu-24.04-amd64 |
|
Revision: 9ccb197 Submitted crossbow builds: ursacomputing/crossbow @ actions-a7f3795a61
|
|
@github-actions crossbow submit verify-rc-source-integration-linux-ubuntu-24.04-amd64 |
|
Revision: df21935 Submitted crossbow builds: ursacomputing/crossbow @ actions-5f5ee06bbb
|
|
@kou I've validated that if we run both tests and integration the job is now successful. I've also validated it failed without the change. |
…rary regardless of ARROW_TEST value (apache#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: apache#48862 Authored-by: Raúl Cumplido <raulcumplido@gmail.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
2 participants
Rationale for this change
Currently if
ARROW_TEST=ONthe 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_integrationlibrary ifARROW_BUILD_INTEGRATION=ONregardless ofARROW_TESTvalue.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
libarrow_c_data_integration.soisn't built withARROW_BUILD_TESTS=ONandARROW_BUILD_INTEGRATION=ON#48862