[PATCH] D69866: [clang-tools-extra] [cmake] Add install target for clangd-indexer

2019-11-05 Thread Mike Pozulp via Phabricator via cfe-commits
mmpozulp created this revision.
mmpozulp added reviewers: ilya-biryukov, mgorny.
Herald added subscribers: cfe-commits, usaxena95, kadircet, arphaman, jkorous, 
MaskRay.
Herald added a project: clang.

Currently if you run 'ninja && ninja install' clangd-indexer will
build but it won't install. This patch makes it install.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D69866

Files:
  clang-tools-extra/clangd/indexer/CMakeLists.txt


Index: clang-tools-extra/clangd/indexer/CMakeLists.txt
===
--- clang-tools-extra/clangd/indexer/CMakeLists.txt
+++ clang-tools-extra/clangd/indexer/CMakeLists.txt
@@ -4,7 +4,7 @@
 Support
 )
 
-add_clang_executable(clangd-indexer
+add_clang_tool(clangd-indexer
   IndexerMain.cpp
   )
 


Index: clang-tools-extra/clangd/indexer/CMakeLists.txt
===
--- clang-tools-extra/clangd/indexer/CMakeLists.txt
+++ clang-tools-extra/clangd/indexer/CMakeLists.txt
@@ -4,7 +4,7 @@
 Support
 )
 
-add_clang_executable(clangd-indexer
+add_clang_tool(clangd-indexer
   IndexerMain.cpp
   )
 
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


[PATCH] D69866: [clang-tools-extra] [cmake] Add install target for clangd-indexer

2019-11-06 Thread Mike Pozulp via Phabricator via cfe-commits
mmpozulp abandoned this revision.
mmpozulp added a comment.

I'm switching to using the background index, so I no longer need to run 
clangd-indexer.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69866/new/

https://reviews.llvm.org/D69866



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits