[PATCH] Teach add_sphinx_target to respect LLVM_INSTALL_TOOLCHAIN_ONLY

Reid Kleckner rnk at google.com
Mon Apr 28 11:31:03 PDT 2014


Go for it.  I was just too lazy to patch it in.  :)


On Sun, Apr 27, 2014 at 1:44 PM, Dan Liew <dan at su-root.co.uk> wrote:

> Ping. I now have commit access. Is it okay for me to commit this?
>
> On 20 April 2014 20:43, Dan Liew <dan at su-root.co.uk> wrote:
> > This is a follow up to my previous patch that Reid committed on my
> > behalf (r206655).
> >
> > I forgot to guard the install command so the docs would of still been
> > installed if the following options were enabled.
> >
> > LLVM_INSTALL_TOOLCHAIN_ONLY
> > LLVM_ENABLE_SPHINX
> > LLVM_BUILD_DOCS
> >
> > The attached patch fixes this.
> >
> > --
> > Dan Liew
> > PhD Student - Imperial College London
>
>
>
> --
> Dan Liew
> PhD Student - Imperial College London
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140428/df6f0788/attachment.html>


More information about the llvm-commits mailing list