[PATCH] Add missing dependency for clang-modernize

Rui Ueyama ruiu at google.com
Tue Oct 15 10:17:53 PDT 2013


Hi revane,

Clang-modernize uses clang-apply-replacements internally but its CMakefile.txt lacked the dependency.

http://llvm-reviews.chandlerc.com/D1938

Files:
  clang-modernize/tool/CMakeLists.txt

Index: clang-modernize/tool/CMakeLists.txt
===================================================================
--- clang-modernize/tool/CMakeLists.txt
+++ clang-modernize/tool/CMakeLists.txt
@@ -30,7 +30,7 @@
   )
 
 add_dependencies(clang-modernize
-  clang-headers
+  clang-headers clang-apply-replacements
   )
 
 target_link_libraries(clang-modernize
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D1938.1.patch
Type: text/x-patch
Size: 355 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131015/4ad8f3c1/attachment.bin>


More information about the llvm-commits mailing list