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

[Dan Liew]

Ping.

Would it be preferable to have a CMake cache variable called
DOCS_PREFIX that is set to ``share/doc/`` by default which the install
rules (doxygen and sphinx) for both clang and llvm use ? This would
prevent inconsistencies if the rules were later modified and allow the
user to customise where the documentation is installed. E.g. Doxygen
install location for LLVM would be

${CMAKE_INSTALL_PREFIX}/${DOCS_PREFIX}/llvm/doxygen

and for LLVM sphinx it would be

${CMAKE_INSTALL_PREFIX}/${DOCS_PREFIX}/llvm/

If so I can rework the patches.


On 20 April 2014 20:53, Dan Liew <dan at su-root.co.uk> wrote:
> When the Doxygen documentation is installed using CMake the install
> location is the same for both LLVM and Clang (assuming clang source is
> in tools/clang). This is bad because some files will over write each
> other and generally make a mess of things.
>
> Attached is a patch for LLVM and a patch for Clang to fix this so LLVM
> and Clang use different installation locations and use an installation
> path that is more similar to the installation location used for the
> Sphinx documentation (recently added in r206655).
>
> Thanks,
> --
> Dan Liew
> PhD Student - Imperial College London



-- 
Dan Liew
PhD Student - Imperial College London