From c7787045e2f75bbb0f0c0dde0fa62782e3435a81 Mon Sep 17 00:00:00 2001 From: Christian Zentgraf Date: Fri, 2 Aug 2024 12:02:52 -0400 Subject: [PATCH] [native] fix openssl not found on Ubuntu MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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. --- presto-native-execution/CMakeLists.txt | 3 --- 1 file changed, 3 deletions(-) diff --git a/presto-native-execution/CMakeLists.txt b/presto-native-execution/CMakeLists.txt index a023b88f0056..78b1d371e536 100644 --- a/presto-native-execution/CMakeLists.txt +++ b/presto-native-execution/CMakeLists.txt @@ -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/openssl@1.1) -set(OPENSSL_ROOT_DIR "${OPT_OPENSSL_DIR}/opt/openssl@1.1") find_package(OpenSSL REQUIRED) find_package(Sodium REQUIRED)