Mir

Merge lp:~desrt/mir/makeinstall-docs into lp:mir/ubuntu

Proposed by Allison Karlitskaya
Status: Superseded
Proposed branch: lp:~desrt/mir/makeinstall-docs
Merge into: lp:mir/ubuntu
Diff against target: 12 lines (+3/-1)
1 file modified
cmake/Doxygen.cmake (+3/-1)
To merge this branch: bzr merge lp:~desrt/mir/makeinstall-docs
Reviewer Review Type Date Requested Status
Alan Griffiths Needs Resubmitting
Kevin DuBois (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+220953@code.launchpad.net

This proposal has been superseded by a proposal from 2014-05-27.

Commit message

build: conditionalise "make install" of docs

Only install the docs if BUILD_DOXYGEN is set (which is the same rule
used to build them in the first place).

Description of the change

build: conditionalise "make install" of docs

Only install the docs if BUILD_DOXYGEN is set (which is the same rule
used to build them in the first place).

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Kevin DuBois (kdub) wrote :

looks good

review: Approve
Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

OK, but please resubmit targeting lp:/mir/devel

review: Needs Resubmitting

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'cmake/Doxygen.cmake'
2--- cmake/Doxygen.cmake 2013-10-21 17:41:35 +0000
3+++ cmake/Doxygen.cmake 2014-05-26 12:44:46 +0000
4@@ -23,5 +23,7 @@
5 COMMAND ${DOXYGEN_EXECUTABLE} ${PROJECT_BINARY_DIR}/Doxyfile
6 SOURCES ${PROJECT_BINARY_DIR}/Doxyfile
7 DEPENDS guides)
8- install(DIRECTORY ${CMAKE_BINARY_DIR}/doc/html DESTINATION ${CMAKE_INSTALL_PREFIX}/share/doc/mir-doc/)
9+ if (BUILD_DOXYGEN)
10+ install(DIRECTORY ${CMAKE_BINARY_DIR}/doc/html DESTINATION ${CMAKE_INSTALL_PREFIX}/share/doc/mir-doc/)
11+ endif()
12 endif()

Subscribers

People subscribed via source and target branches