[llvm] r297545 - Revert r297516 - Respect CMAKE_INSTALL_MANDIR for sphinx generated manpages

Eric Fiselier via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 10 18:24:13 PST 2017


Author: ericwf
Date: Fri Mar 10 20:24:13 2017
New Revision: 297545

URL: http://llvm.org/viewvc/llvm-project?rev=297545&view=rev
Log:
Revert r297516 - Respect CMAKE_INSTALL_MANDIR for sphinx generated manpages

When CMAKE_INSTALL_MANDIR isn't defined it ends up attempting to install
the man pages under "/man1" and we really don't want to accidentally install
stuff at the filesystem root.

Modified:
    llvm/trunk/cmake/modules/AddSphinxTarget.cmake

Modified: llvm/trunk/cmake/modules/AddSphinxTarget.cmake
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/cmake/modules/AddSphinxTarget.cmake?rev=297545&r1=297544&r2=297545&view=diff
==============================================================================
--- llvm/trunk/cmake/modules/AddSphinxTarget.cmake (original)
+++ llvm/trunk/cmake/modules/AddSphinxTarget.cmake Fri Mar 10 20:24:13 2017
@@ -51,7 +51,7 @@ function (add_sphinx_target builder proj
         # FIXME: We might not ship all the tools that these man pages describe
         install(DIRECTORY "${SPHINX_BUILD_DIR}/" # Slash indicates contents of
                 COMPONENT "${project}-sphinx-man"
-                DESTINATION ${CMAKE_INSTALL_MANDIR}/man1)
+                DESTINATION share/man/man1)
 
       elseif (builder STREQUAL html)
         string(TOUPPER "${project}" project_upper)




More information about the llvm-commits mailing list