[PATCH] D91001: [clang-tidy] Install run-clang-tidy.py as run-clang-tidy

Florian Schmaus via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Nov 7 04:23:29 PST 2020


Flow created this revision.
Flow added a project: clang-tools-extra.
Herald added subscribers: cfe-commits, xazax.hun, mgorny.
Herald added a project: clang.
Flow requested review of this revision.

Installed scripts in PATH usually do not carry a filename extension,
since there is no need to know that this is a Python script. For
example Debian and Ubuntu already install this script as
'run-clang-tidy' [1] and hence build systems like Meson also look for
this name first [2].

This changes was suggested by Sylvestre Ledru [3].

1: https://salsa.debian.org/pkg-llvm-team/llvm-toolchain/-/blob/60aefb14171ab5c3867a0081844b507fc9f6e015/debian/clang-tidy-X.Y.links.in#L2
2: https://github.com/mesonbuild/meson/blob/b6dc4d5e5c6e838de0b52e62d982ba2547eb366d/mesonbuild/scripts/clangtidy.py#L44
3: https://reviews.llvm.org/D90972#2380640


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D91001

Files:
  clang-tools-extra/clang-tidy/tool/CMakeLists.txt


Index: clang-tools-extra/clang-tidy/tool/CMakeLists.txt
===================================================================
--- clang-tools-extra/clang-tidy/tool/CMakeLists.txt
+++ clang-tools-extra/clang-tidy/tool/CMakeLists.txt
@@ -56,4 +56,5 @@
   COMPONENT clang-tidy)
 install(PROGRAMS run-clang-tidy.py
   DESTINATION bin
-  COMPONENT clang-tidy)
+  COMPONENT clang-tidy
+  RENAME run-clang-tidy)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91001.303634.patch
Type: text/x-patch
Size: 403 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20201107/28787f82/attachment.bin>


More information about the cfe-commits mailing list