[llvm] 2978b10 - [clangd] Fix remote index build on macOS
Kirill Bobyrev via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 31 05:03:18 PDT 2020
Author: Kirill Bobyrev
Date: 2020-07-31T14:02:24+02:00
New Revision: 2978b10aa164f692d48041327e27b2811649a233
URL: https://github.com/llvm/llvm-project/commit/2978b10aa164f692d48041327e27b2811649a233
DIFF: https://github.com/llvm/llvm-project/commit/2978b10aa164f692d48041327e27b2811649a233.diff
LOG: [clangd] Fix remote index build on macOS
macOS builds suddenly started failing:
https://github.com/kirillbobyrev/indexing-tools/runs/925090879
This patch makes use of imported libraries and fixes builds for macOS.
Landing this without a review since the patch is quite straightforward
and I've been testing it on my local macOS machine for a while.
Differential Revision: https://reviews.llvm.org/D84928
Added:
Modified:
llvm/cmake/modules/FindGRPC.cmake
Removed:
################################################################################
diff --git a/llvm/cmake/modules/FindGRPC.cmake b/llvm/cmake/modules/FindGRPC.cmake
index 8a0ca593b2f4..70d67ceb791d 100644
--- a/llvm/cmake/modules/FindGRPC.cmake
+++ b/llvm/cmake/modules/FindGRPC.cmake
@@ -45,11 +45,25 @@ else()
# system path.
if (GRPC_HOMEBREW_RETURN_CODE EQUAL "0")
include_directories(${GRPC_HOMEBREW_PATH}/include)
- link_directories(${GRPC_HOMEBREW_PATH}/lib)
+ find_library(GRPC_LIBRARY
+ grpc++
+ PATHS ${GRPC_HOMEBREW_PATH}/lib
+ NO_DEFAULT_PATH
+ REQUIRED)
+ add_library(grpc++ UNKNOWN IMPORTED GLOBAL)
+ set_target_properties(grpc++ PROPERTIES
+ IMPORTED_LOCATION ${GRPC_LIBRARY})
endif()
if (PROTOBUF_HOMEBREW_RETURN_CODE EQUAL "0")
include_directories(${PROTOBUF_HOMEBREW_PATH}/include)
- link_directories(${PROTOBUF_HOMEBREW_PATH}/lib)
+ find_library(PROTOBUF_LIBRARY
+ protobuf
+ PATHS ${PROTOBUF_HOMEBREW_PATH}/lib
+ NO_DEFAULT_PATH
+ REQUIRED)
+ add_library(protobuf UNKNOWN IMPORTED GLOBAL)
+ set_target_properties(protobuf PROPERTIES
+ IMPORTED_LOCATION ${PROTOBUF_LIBRARY})
endif()
endif()
endif()
More information about the llvm-commits
mailing list