[PATCH] D31687: CMake: Fix sphinx build with standalone clang

Chris Bieneman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 5 13:13:54 PDT 2017


beanz added a comment.

I think putting this into AddSphinxTarget.cmake would be a better way to handle it. Also I have one small inline comment.



================
Comment at: CMakeLists.txt:177
+    find_package(Sphinx REQUIRED)
+    if (LLVM_BUILD_DOCS)
+      add_custom_target(sphinx ALL)
----------------
If tis code can be run more than once you also need to add `AND NOT TARGET sphinx` to the condition here.


https://reviews.llvm.org/D31687





More information about the llvm-commits mailing list