Skip to content

Commit

Permalink
GH-36707: [C++] Use ARROW_PACKAGE_PREFIX for OPENSSL_ROOT_DIR too (#3…
Browse files Browse the repository at this point in the history
…6710)

### Rationale for this change

In general, a CMake package uses `${PACKAGE}_ROOT` variable to detect `PACKAGE` but `FindOpenSSL.cmake` uses `OPENSSL_ROOT_DIR` not `OpenSSL_ROOT`.

### What changes are included in this PR?

Set `OPENSSL_ROOT_DIR` explicitly.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes.
* Closes: #36707

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Raúl Cumplido <raulcumplido@gmail.com>
  • Loading branch information
kou authored and raulcd committed Jul 17, 2023
1 parent 1d5f4cd commit 453965d
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions cpp/cmake_modules/ThirdpartyToolchain.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -138,6 +138,9 @@ if(ARROW_PACKAGE_PREFIX)
if(NOT ENV{Boost_ROOT})
set(ENV{Boost_ROOT} ${ARROW_PACKAGE_PREFIX})
endif()
if(NOT DEFINED OPENSSL_ROOT_DIR)
set(OPENSSL_ROOT_DIR ${ARROW_PACKAGE_PREFIX})
endif()
endif()

# For each dependency, set dependency source to global default, if unset
Expand Down

0 comments on commit 453965d

Please sign in to comment.