[clang-tidy] install helper scripts in CMake build

[Eugene.Zelenko]

Summary

Scripts are installed in same location as clang-fromat ones, so I think will be good idea to not create dedicated directory.

I checked this patch on my own build on RHEL 6.

Please check it in if it's OK, because I don't have SVN write access.

I think will be good idea to backport this patch to 3.7 release branch.

Probably same should be done for configure build.

[alexfh]

Not sure whether the scripts should go to share/clang or some other place. Sylvestre should know what's the common practice. Also, configure builds are still used for building packages, AFAIU, so they have to be modified as well (preferably, in the same patch).

[Eugene.Zelenko]

On my understanding share contains architecture-independent files.

clang-format files installation is also part of CMake build only.

[sylvestre.ledru]

share/clang is fine, thanks.

[alexfh]

In D12700#246948, @sylvestre.ledru wrote:

share/clang is fine, thanks.

Then we need to make a similar change to the configure build. Eugene, can you do this?

[Eugene.Zelenko]

I added installation in configure. See also D12919 for further configure improvements idea.

[alexfh]

LG. Thanks!

[probinson]

Not sure what the exact problem is, but we still have one bot using configure/make, and it dies with this on the install step unless we use -j1. I know the hand-written Makefiles are deprecated and we've worked around it, but I thought I should mention it.

[Eugene.Zelenko]

I think will be good idea to ask specialist in configure build to look on make file. I used other Clang or LLVM make file as example. Frankly, I don't see anything bad with it.

However this fix was backported to 3.7.x and configure build is still supported there.

If root cause will not be found, it' OK for me to remove installation from configure build, since I use CMake one for my build.

[probinson]

The problem is that making the install-local target will try to create the PROJ_sharedir directory in parallel with trying to copy DESTFILES to that directory. You need to set up the dependencies such that the directory is guaranteed to be created first. If you move the PROJ_sharedir dependency from the install-local target to the DESTFILES target that should solve it:

$(DESTFILES): $(SRCFILES) $(PROJ_sharedir)
...
install-local:: $(DESTFILES)

clang-tools-extra/trunk/clang-tidy/tool/CMakeLists.txt

target_link_libraries(clang-tidy
   clangTidyModernizeModule
   clangTidyReadabilityModule
   clangTooling
   )
 
 install(TARGETS clang-tidy
   RUNTIME DESTINATION bin)
 
+install(PROGRAMS clang-tidy-diff.py DESTINATION share/clang)
+install(PROGRAMS run-clang-tidy.py DESTINATION share/clang)

clang-tools-extra/trunk/clang-tidy/tool/Makefile

USEDLIBS = clangTidy.a clangTidyLLVMModule.a clangTidyGoogleModule.a \
      clangTidyUtils.a clangStaticAnalyzerFrontend.a \
      clangStaticAnalyzerCheckers.a clangStaticAnalyzerCore.a \
      clangFormat.a clangASTMatchers.a clangTooling.a clangToolingCore.a \
      clangFrontend.a clangSerialization.a clangDriver.a clangParse.a \
      clangSema.a clangAnalysis.a clangRewriteFrontend.a clangRewrite.a \
      clangEdit.a clangAST.a clangLex.a clangBasic.a
 
 include $(CLANG_LEVEL)/Makefile
+
+PROJ_sharedir := $(DESTDIR)$(PROJ_prefix)/share/clang
+
+FILESLIST := $(notdir $(wildcard $(PROJ_SRC_DIR)/*.py))
+
+SRCFILES := $(addprefix $(PROJ_SRC_DIR)/, $(FILESLIST))
+DESTFILES := $(addprefix $(PROJ_sharedir)/, $(FILESLIST))
+
+$(PROJ_sharedir):
+  $(Echo) Making install directory: $@
+  $(Verb) $(MKDIR) $@
+
+$(DESTFILES): $(SRCFILES)
+
+$(PROJ_sharedir)/%.py: $(PROJ_SRC_DIR)/%.py
+  $(Echo) Installing script file: $(notdir $<)
+  $(Verb) $(ScriptInstall) $< $(PROJ_sharedir)
+
+install-local:: $(PROJ_sharedir) $(DESTFILES)