[PATCH] D73236: [clang-tidy] Add clang-tidy headers to clang distribution

Shoaib Meenai via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 22 15:40:21 PST 2020


smeenai added a comment.

Can you upload the patch with context (`git diff -U9999` if you're using the web interface or via arcanist)?

You'll want to add install targets for this, by adding a `COMPONENT` to the `install` command and referencing that in the target. See https://reviews.llvm.org/source/llvm-github/browse/master/clang/CMakeLists.txt;b4a99a061f517e60985667e39519f60186cbb469$438-446 for an example, though in your case the custom target doesn't need any dependencies.



================
Comment at: clang-tools-extra/CMakeLists.txt:37
+  install(DIRECTORY clang-tidy
+    DESTINATION include
+    FILES_MATCHING
----------------
Is it possible to set this to `include/clang-tidy` and then move the block into `clang-tidy/CMakeLists.txt`?


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D73236





More information about the cfe-commits mailing list