<div dir="ltr">Ok, makes sense.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 14, 2014 at 2:40 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"><div class="">On 14 August 2014 21:40, Sean Silva <<a href="mailto:chisophugis@gmail.com">chisophugis@gmail.com</a>> wrote:<br>

><br>
><br>
><br>
> On Thu, Aug 14, 2014 at 4:57 AM, Dan Liew <<a href="mailto:dan@su-root.co.uk">dan@su-root.co.uk</a>> wrote:<br>
>><br>
>> Author: delcypher<br>
>> Date: Thu Aug 14 06:57:13 2014<br>
>> New Revision: 215634<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=215634&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=215634&view=rev</a><br>
>> Log:<br>
>> Add SPHINX_WARNINGS_AS_ERRORS CMake option to allow warnings to not be<br>
>> treated as errors (which is still the default).<br>
><br>
><br>
> This seems to contradict the patch, which seems to make -W the default<br>
> (SPHINX_WARNINGS_AS_ERRORS=ON by default).<br>
<br>
</div>Not really. Before sphinx warnings were always treated as errors. Now<br>
sphinx warnings can be ignored by setting<br>
SPHINX_WARNINGS_AS_ERRORS=OFF. So we gain the ability to "allow<br>
warnings to not be treated as errors".<br>
<br>
Admittedly I could of worded the commit log a lot better so it doesn't<br>
sound like I'm contradicting myself.<br>
</blockquote></div><br></div>