[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