<div dir="ltr">Go for it.  I was just too lazy to patch it in.  :)</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Apr 27, 2014 at 1:44 PM, 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">Ping. I now have commit access. Is it okay for me to commit this?<br>
<div class="HOEnZb"><div class="h5"><br>
On 20 April 2014 20:43, Dan Liew <<a href="mailto:dan@su-root.co.uk">dan@su-root.co.uk</a>> wrote:<br>
> This is a follow up to my previous patch that Reid committed on my<br>
> behalf (r206655).<br>
><br>
> I forgot to guard the install command so the docs would of still been<br>
> installed if the following options were enabled.<br>
><br>
> LLVM_INSTALL_TOOLCHAIN_ONLY<br>
> LLVM_ENABLE_SPHINX<br>
> LLVM_BUILD_DOCS<br>
><br>
> The attached patch fixes this.<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>