[PATCH] D24935: [cmake] Support overriding remaining HTML doc install directories
Chris Bieneman via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 27 10:52:26 PDT 2016
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
More information about the llvm-commits
mailing list