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

[Reid Kleckner]

Should this be in share/[llvm|clang]/doc or share/doc/[llvm|clang] ?  We
already have stuff in the former.


On Mon, Apr 28, 2014 at 2:32 AM, Dan Liew <dan at su-root.co.uk> wrote:

> 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
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140501/0689fe6f/attachment.html>