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)