[PATCH] Fix installation location of Doxygen generated documentation in CMake

[Dan Liew]

On 1 May 2014 19:25, Reid Kleckner <rnk at google.com> wrote:
> Should this be in share/[llvm|clang]/doc or share/doc/[llvm|clang] ?  We
> already have stuff in the former.

I don't see anything installed to ``share/[llvm|clang]/doc`` in the
CMake build. Are you referring to the install location for to
autoconf/make build system?

Currently, the install locations for documentation using CMake are:

* Sphinx html

${CMAKE_INSTALL_PREFIX}/share/doc/llvm/
${CMAKE_INSTALL_PREFIX}/share/doc/clang/
${CMAKE_INSTALL_PREFIX}/share/doc/lld/

* LLVM Sphinx man pages:

${CMAKE_INSTALL_PREFIX}/share/man/man1

* Doxygen:
Clang and LLVM both use the same path below (not a good idea!)

${CMAKE_INSTALL_PREFIX}/docs/html/html

My patch is proposing to change the Doxygen install location for CMake to

${CMAKE_INSTALL_PREFIX}/share/doc/llvm/doxygen

${CMAKE_INSTALL_PREFIX}/share/doc/clang/doxygen

which is inside the Sphinx directory. Currently there is no conflict
but maybe it ought to be outside the sphinx install directory?

Regarding the "share/doc/llvm" vs "share/llvm/doc" issue, as I
mentioned earlier it might be more sensible to provide a CMake cache
variable that users can set so the choose what prefix to use for the
documentation.