<div dir="ltr"><div>Sorry for the delay.</div><div><br></div><div>This is looking really good. I would like to remove the linkcheck target just on the principle that the build should not go accessing external websites.</div>
<div><br></div><div>Also, fixing those broken links is a "patches welcome" situation that many new contributors will actually bite on. That's not an argument for not fixing the links; it's an argument for not worrying too much about not fixing them. Rather than fix trivial mechanical issues like broken links, people that have more experience with LLVM would serve the community better by actually contributing *content* <<a href="http://llvm.org/docs/SphinxQuickstartTemplate.html">http://llvm.org/docs/SphinxQuickstartTemplate.html</a>> covering a specific area that they know about.</div>
<div><br></div><div>You should send 0002 in a separate patch thread. I don't really have the knowledge to sign off on it. I think you'll want to CC chapuni or chandler.</div><div><br></div><div>-- Sean Silva</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Apr 16, 2014 at 11:50 AM, Dan Liew <span dir="ltr"><<a href="mailto:dan@su-root.co.uk" target="_blank">dan@su-root.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Okay here is my second attempt at this. Rather than explain the<br>
differences, here's how to use it<br>
<br>
Build with the following CMake options enabled<br>
<br>
LLVM_ENABLE_SPHINX<br>
LLVM_BUILD_DOCS<br>
<br>
This will enable and build by default the following targets<br>
docs-llvm-html, docs-llvm-man.<br>
<br>
If you run ``make install`` then the documentation will be installed<br>
in the followings locations<br>
<br>
HTML files are installed to<br>
<br>
${CMAKE_INSTALL_PREFIX}/share/doc/${CMAKE_PROJECT_NAME}/html/<br>
<br>
Man pages are installed to<br>
<br>
${CMAKE_INSTALL_PREFIX}/share/man/man1/<br>
<br>
These locations are correct for my particular Linux install but I'm<br>
not sure about OSX or Windows.<br>
<br>
I've put the add_sphinx_target() CMake function in a separate file so<br>
it can be included by other projects (e.g. Clang). It just needs to be<br>
called by CMakeLists.txt file within a directory containing Sphinx<br>
documentation.<br>
<br>
I've tested this with make and ninja under Linux and it seems to work fine.<br>
<br>
I've also attached a second patch that makes configure fail if<br>
LLVM_ENABLE_DOXYGEN was enabled but doxygen was not found.<br>
<br>
Thanks,<br>
Dan.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
On 16 April 2014 02:02, Dan Liew <<a href="mailto:dan@su-root.co.uk">dan@su-root.co.uk</a>> wrote:<br>
>> Maybe it's because of the :program: tag?<br>
><br>
> I think it's actually the docs/conf.py configuration file that<br>
> populates the man_pages [1] variable based on the contents of<br>
> docs/CommandGuide directory<br>
><br>
> The program:: directive seems to tell sphinx that following option::<br>
> directives refer to that tool [2]<br>
><br>
> [1] <a href="http://sphinx-doc.org/config.html#confval-man_pages" target="_blank">http://sphinx-doc.org/config.html#confval-man_pages</a><br>
> [2] <a href="http://sphinx-doc.org/domains.html#directive-program" target="_blank">http://sphinx-doc.org/domains.html#directive-program</a><br>
><br>
> --<br>
> Dan Liew<br>
> PhD Student - Imperial College London<br>
<br>
<br>
<br>
--<br>
Dan Liew<br>
PhD Student - Imperial College London<br>
</div></div></blockquote></div><br></div>