[PATCH] D23757: cmake: Support overriding Sphinx HTML doc install directory

[Michał Górny via llvm-commits]

mgorny added inline comments.

================
Comment at: docs/CMake.rst:461
@@ +460,3 @@
+  The path to install Sphinx-generated HTML documentation to. Defaults to
+  /usr/share/doc/llvm/html.
+
----------------
delcypher wrote:
> This default is incorrect. The default is actually `share/doc/llvm/html`. It does not contain the `CMAKE_INSTALL_PREFIX`.
> 
> Your text probably ought to say something like:
> 
> ```
> The path to install Sphinx-generated HTML documentation to. This path relative to the CMAKE_INSTALL_PREFIX and by default
> is `share/doc/llvm/html`.
> ```
Well, this is not 100% correct since it is relative only when it's not absolute ;-). I'll try to find a good way to express it.


https://reviews.llvm.org/D23757