From 20d8acd89f5ebf87295e08ed10e2f94cb03d57d0 Mon Sep 17 00:00:00 2001 From: Sutou Kouhei Date: Fri, 21 Feb 2025 05:31:47 +0900 Subject: [PATCH] GH-45568: [C++][Parquet][CMake] Enable zlib automatically when Thrift is needed (#45569) ### Rationale for this change Required dependencies checks must be done automatically. ### What changes are included in this PR? * Fix variable name * Fix check order ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * GitHub Issue: #45568 Authored-by: Sutou Kouhei Signed-off-by: Sutou Kouhei --- cpp/cmake_modules/ThirdpartyToolchain.cmake | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/cpp/cmake_modules/ThirdpartyToolchain.cmake b/cpp/cmake_modules/ThirdpartyToolchain.cmake index 279d2c0fdfe63..9b531c8d5a4b6 100644 --- a/cpp/cmake_modules/ThirdpartyToolchain.cmake +++ b/cpp/cmake_modules/ThirdpartyToolchain.cmake @@ -386,14 +386,14 @@ if(ARROW_WITH_OPENTELEMETRY) set(ARROW_WITH_PROTOBUF ON) endif() -if(ARROW_THRIFT) - set(ARROW_WITH_ZLIB ON) -endif() - if(ARROW_PARQUET) set(ARROW_WITH_THRIFT ON) endif() +if(ARROW_WITH_THRIFT) + set(ARROW_WITH_ZLIB ON) +endif() + if(ARROW_FLIGHT) set(ARROW_WITH_GRPC ON) endif()