From: Kirill Bobyrev <kbobyrev@google.com>
Date: Mon, 26 Oct 2020 19:37:43 +0000 (+0100)
Subject: [clangd] Add back dependency on proto generated targets
X-Git-Tag: llvmorg-13-init~8127
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d071bba9a4477290d0b164a2d338fb22ad2d3c3d;p=platform%2Fupstream%2Fllvm.git

[clangd] Add back dependency on proto generated targets

Previous attempts:

* 15f6bad6d74a993e366c8fc93a9c91f213ac6bc3
* 58d0ef2d0466a893ab400f6a9829057b9d851038

The combination results in both link- and build-time dependency which is
the desired behavior.
---

diff --git a/clang-tools-extra/clangd/index/remote/CMakeLists.txt b/clang-tools-extra/clangd/index/remote/CMakeLists.txt
index 5a9f49a..971af20 100644
--- a/clang-tools-extra/clangd/index/remote/CMakeLists.txt
+++ b/clang-tools-extra/clangd/index/remote/CMakeLists.txt
@@ -3,9 +3,10 @@ if (CLANGD_ENABLE_REMOTE)
   generate_protos(RemoteIndexProto "Index.proto")
   target_link_libraries(RemoteIndexServiceProto
 
-    PRIVATE
+    PUBLIC
     RemoteIndexProto
     )
+  add_dependencies(RemoteIndexServiceProto RemoteIndexProto)
   include_directories(${CMAKE_CURRENT_BINARY_DIR})
   include_directories(${CMAKE_CURRENT_SOURCE_DIR}/../../)