[PATCH] D24935: [cmake] Support overriding remaining HTML doc install directories

[Chris Bieneman via llvm-commits]

beanz accepted this revision.
beanz added a reviewer: beanz.
beanz added a comment.
This revision is now accepted and ready to land.

Small nit below, otherwise LGTM! Thanks!


================
Comment at: docs/CMakeLists.txt:99
@@ -98,1 +98,3 @@
+    install(DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/doxygen/html/.
+      DESTINATION "${LLVM_INSTALL_DOXYGEN_HTML_DIR}")
   endif()
----------------
Can you add a `COMPONENT` to the install commands, maybe something like "doxygen-html"?

================
Comment at: docs/CMakeLists.txt:162
@@ -159,1 +161,3 @@
+    install(DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/ocamldoc/html/.
+      DESTINATION "${LLVM_INSTALL_OCAMLDOC_HTML_DIR}")
   endif()
----------------
Here too.


https://reviews.llvm.org/D24935