[PATCH] D23832: cmake: Add ordering dep between HTML Sphinx docs and manpages

[Michał Górny via cfe-commits]

mgorny created this revision.
mgorny added reviewers: rnk, axw.
mgorny added a subscriber: cfe-commits.

Add a dependency between HTML & manpage Sphinx targets to prevent two instances 
of Sphinx from running in parallel, and therefore solves race conditions 
reusing the same doctree directory.

Bug: https://llvm.org/bugs/show_bug.cgi?id=23781

Also submitted for llvm: https://reviews.llvm.org/D23755

https://reviews.llvm.org/D23832

Files:
  docs/CMakeLists.txt

Index: docs/CMakeLists.txt
===================================================================
--- docs/CMakeLists.txt
+++ docs/CMakeLists.txt
@@ -102,6 +102,9 @@
     endif()
     if (${SPHINX_OUTPUT_MAN})
       add_sphinx_target(man clang)
+      if (${SPHINX_OUTPUT_HTML})
+        add_dependencies(docs-clang-html docs-clang-man)
+      endif()
     endif()
   endif()
 endif()



Index: docs/CMakeLists.txt
===================================================================
--- docs/CMakeLists.txt
+++ docs/CMakeLists.txt
@@ -102,6 +102,9 @@
     endif()
     if (${SPHINX_OUTPUT_MAN})
       add_sphinx_target(man clang)
+      if (${SPHINX_OUTPUT_HTML})
+        add_dependencies(docs-clang-html docs-clang-man)
+      endif()
     endif()
   endif()
 endif()

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits