Skip to content

Commit

Permalink
[native] fix openssl not found on Ubuntu
Browse files Browse the repository at this point in the history
The OPENSSL_ROOT_DIR points to a path that does not exist.
This was removed on Velox. It doesn’t cause a problem on MacOS or CentOS but
on Ubuntu it will prevent the build by throwing

Could NOT find OpenSSL, try to set the path to OpenSSL root folder in the
  system variable OPENSSL_ROOT_DIR (missing: OPENSSL_CRYPTO_LIBRARY
  OPENSSL_INCLUDE_DIR)

Removing setting the OPENSSL_ROOT_DIR fixes the issue.
  • Loading branch information
czentgr authored and majetideepak committed Sep 4, 2024
1 parent b1bf2b9 commit c778704
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions presto-native-execution/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -170,9 +170,6 @@ if(BZIP2_FOUND)
endif()
include_directories(SYSTEM ${FOLLY_INCLUDE_DIRS})

# Include third party header files
find_path(OPT_OPENSSL_DIR NAMES opt/[email protected])
set(OPENSSL_ROOT_DIR "${OPT_OPENSSL_DIR}/opt/[email protected]")
find_package(OpenSSL REQUIRED)

find_package(Sodium REQUIRED)
Expand Down

0 comments on commit c778704

Please sign in to comment.