[arrow] branch master updated: ARROW-8689: [C++] Fix linking S3FS benchmarks

[ap...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new ba5efef  ARROW-8689: [C++] Fix linking S3FS benchmarks
ba5efef is described below

commit ba5efeffeeb1db4d48102be14125ef7d224d308b
Author: Antoine Pitrou <an...@python.org>
AuthorDate: Mon May 4 16:48:57 2020 +0200

    ARROW-8689: [C++] Fix linking S3FS benchmarks
    
    Closes #7094 from pitrou/ARROW-8689-s3fs-benchmark-failing
    
    Authored-by: Antoine Pitrou <an...@python.org>
    Signed-off-by: Antoine Pitrou <an...@python.org>
---
 cpp/cmake_modules/ThirdpartyToolchain.cmake | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/cpp/cmake_modules/ThirdpartyToolchain.cmake b/cpp/cmake_modules/ThirdpartyToolchain.cmake
index 4164187..2ca5d64 100644
--- a/cpp/cmake_modules/ThirdpartyToolchain.cmake
+++ b/cpp/cmake_modules/ThirdpartyToolchain.cmake
@@ -703,10 +703,12 @@ endif()
 
 # - Parquet requires boost only with gcc 4.8 (because of missing std::regex).
 # - Gandiva has a compile-time (header-only) dependency on Boost, not runtime.
-# - Tests and Flight benchmarks need Boost at runtime.
+# - Tests need Boost at runtime.
+# - 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_S3 AND ARROW_BUILD_BENCHMARKS)
    OR ARROW_GANDIVA
    OR (ARROW_WITH_THRIFT AND Thrift_SOURCE STREQUAL "BUNDLED")
    OR (ARROW_PARQUET