From: Antonio Ospite <ospite@studenti.unina.it>
Date: Wed, 28 Mar 2012 11:08:23 +0000 (+0200)
Subject: doc: fix the "custom install targets" to handle DESTDIR
X-Git-Tag: v0.1.2~1
X-Git-Url: https://git.ao2.it/libam7xxx.git/commitdiff_plain/67fd00b35aabe932cff75d1945b26db7020d1ec6?ds=sidebyside;hp=0fa2d27df5433816e7e659bd1115d2fff42288b0

doc: fix the "custom install targets" to handle DESTDIR

Stuff has to be installed/copied in an actual "install" target for Cmake
to handle DESTDIR sanely, a custom target is not enough.

So the story now is:

  - add a component-local install rule

  - make the "custom install target" invoke it

  - make sure the custom target has been executed at install time

Maybe all this is not even worth the effort anymore and the "custom
install targets" could go away altogether, their point was to have
_them_ invoke at install time end exploit the dependencies chain.
---

diff --git a/doc/CMakeLists.txt b/doc/CMakeLists.txt
index 5a7d9cb..6ccf100 100644
--- a/doc/CMakeLists.txt
+++ b/doc/CMakeLists.txt
@@ -10,13 +10,15 @@ if(DOXYGEN_FOUND)
   )
 
   add_custom_target(install-doc
-    ${CMAKE_COMMAND} -E copy_directory
-    ${DOC_OUTPUT_PATH}/html
-    "${CMAKE_INSTALL_PREFIX}/share/doc/${CMAKE_PROJECT_NAME}/html"
-  )
+    COMMAND ${CMAKE_COMMAND} -DCOMPONENT=doc -P ${CMAKE_BINARY_DIR}/cmake_install.cmake
+    COMMENT "Installing API documentation")
   add_dependencies(install-doc doc)
 
-  install(CODE "execute_process(COMMAND ${CMAKE_BUILD_TOOL} install-doc)")
+  install(CODE "execute_process(COMMAND ${CMAKE_BUILD_TOOL} doc)")
+
+  install(DIRECTORY ${DOC_OUTPUT_PATH}/html
+    DESTINATION "${CMAKE_INSTALL_PREFIX}/share/doc/${CMAKE_PROJECT_NAME}"
+    COMPONENT doc)
 
 endif(DOXYGEN_FOUND)
 
diff --git a/doc/man/CMakeLists.txt b/doc/man/CMakeLists.txt
index 17549bf..ed46999 100644
--- a/doc/man/CMakeLists.txt
+++ b/doc/man/CMakeLists.txt
@@ -9,17 +9,16 @@ if(ASCIIDOC_FOUND)
   )
 
   add_custom_target(install-manpages
-    ${CMAKE_COMMAND} -E make_directory
-      "${CMAKE_INSTALL_PREFIX}/share/man/man1/"
-    COMMAND ${CMAKE_COMMAND} -E copy
-      ${DOC_OUTPUT_PATH}/man/am7xxx-play.1
-      "${CMAKE_INSTALL_PREFIX}/share/man/man1/"
-    COMMAND ${CMAKE_COMMAND} -E copy
-      ${DOC_OUTPUT_PATH}/man/picoproj.1
-      "${CMAKE_INSTALL_PREFIX}/share/man/man1/"
-  )
+    COMMAND ${CMAKE_COMMAND} -DCOMPONENT=manpages -P ${CMAKE_BINARY_DIR}/cmake_install.cmake
+    COMMENT "Installing manpages")
   add_dependencies(install-manpages manpages)
 
-  install(CODE "execute_process(COMMAND ${CMAKE_BUILD_TOOL} install-manpages)")
+  install(CODE "execute_process(COMMAND ${CMAKE_BUILD_TOOL} manpages)")
+
+  install(FILES
+    ${DOC_OUTPUT_PATH}/man/am7xxx-play.1
+    ${DOC_OUTPUT_PATH}/man/picoproj.1
+    DESTINATION "${CMAKE_INSTALL_PREFIX}/share/man/man1/"
+    COMPONENT manpages)
 
 endif(ASCIIDOC_FOUND)