Skip to content

Commit

Permalink
GH-45568: [C++][Parquet][CMake] Enable zlib automatically when Thrift…
Browse files Browse the repository at this point in the history
… 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 <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
  • Loading branch information
kou authored Feb 20, 2025
1 parent 59fd94f commit 20d8acd
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions cpp/cmake_modules/ThirdpartyToolchain.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -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()
Expand Down

0 comments on commit 20d8acd

Please sign in to comment.