[PATCH] CMake: enable building the clang-format vs plugin

Manuel Klimek klimek at google.com
Tue Dec 3 00:44:41 PST 2013



================
Comment at: tools/clang-format-vs/CMakeLists.txt:6
@@ +5,3 @@
+      "${LLVM_TOOLS_BINARY_DIR}/clang-format.exe"
+      "${CMAKE_CURRENT_SOURCE_DIR}/ClangFormat/clang-format.exe"
+      DEPENDS clang-format)
----------------
One more thought - the source directory might not be writable, usually it's a best practice to use the output directory for this and a build variable to inject the path (don't know how exactly that works with MSVC, so I think this is fine for now).


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



More information about the cfe-commits mailing list