You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2023/04/06 02:16:03 UTC
[kudu] branch master updated: [build] Fix build issue on MacOS
This is an automated email from the ASF dual-hosted git repository.
alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git
The following commit(s) were added to refs/heads/master by this push:
new f13f6580c [build] Fix build issue on MacOS
f13f6580c is described below
commit f13f6580c0588d8596f76e8ad9e39feba7a13783
Author: Attila Bukor <ab...@apache.org>
AuthorDate: Wed Apr 5 16:26:28 2023 +0200
[build] Fix build issue on MacOS
Building Kudu on MacOS was failing due to a linking issue. Specifically,
BIO_* functions weren't reachable from jwt_test_certs. It's unclear to
me how exactly it worked on Linux, I suspect the linker picks up OpenSSL
libraries automatically on Linux. Anyway, this commit fix the build by
adding the correct dependencies to jwt_test_certs.
Change-Id: Ifded17faafe99d8bbb896dc65298a8bd4c38854f
Reviewed-on: http://gerrit.cloudera.org:8080/19692
Tested-by: Attila Bukor <ab...@apache.org>
Reviewed-by: Marton Greber <gr...@gmail.com>
Reviewed-by: Alexey Serbin <al...@apache.org>
---
src/kudu/util/CMakeLists.txt | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/src/kudu/util/CMakeLists.txt b/src/kudu/util/CMakeLists.txt
index 77e01fe95..b6975a6d9 100644
--- a/src/kudu/util/CMakeLists.txt
+++ b/src/kudu/util/CMakeLists.txt
@@ -364,12 +364,6 @@ target_link_libraries(mini_oidc
kudu_util
jwt_test_certs)
-#######################################
-# jwt_test_certs
-#######################################
-add_library(jwt_test_certs jwt_test_certs.cc)
-target_link_libraries(jwt_test_certs)
-
#######################################
# kudu_jwt_util
#######################################
@@ -380,6 +374,13 @@ target_link_libraries(kudu_jwt_util
kudu_curl_util
kudu_util)
+#######################################
+# jwt_test_certs
+#######################################
+add_library(jwt_test_certs jwt_test_certs.cc)
+target_link_libraries(jwt_test_certs
+ kudu_jwt_util)
+
#######################################
# kudu_cloud_util
#######################################