<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 7, 2014 at 3:47 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"><div class="">>> >> @Sean Silva : I know you disagree with having Sphinx warnings as<br>
>> >> errors but I think for the buildbot that this is probably fine.<br>
<br>
</div><snip><br>
<div class=""><br>
>> I think we could experiment with this. We could start with the patch<br>
>> as it is and if devs find it irritating<br>
><br>
><br>
> An email from a buildbot is not really an invitation for a dialogue about<br>
> whether the dev finds it irritating. Before you set up a bot, I would<br>
> recommend that you start out by personally emailing the devs about these<br>
> issues. That is an easy way to find out if e.g. they usually need help<br>
> understanding what Sphinx is warning about and how to fix it.<br>
><br>
> Also, unlike a bot, you have a gut feeling about how annoying you are being<br>
> when emailing someone about something, which lets you estimate the<br>
> "annoyance factor" without having to rely on developers taking action to<br>
> inform "the bot operator" that the bot is annoying them (or even worse, just<br>
> silently and/or subconsciously contributing less to the documentation).<br>
<br>
</div>Although this does sound like a good idea I'm not sure I really have<br>
the free time to dedicate to doing that right now (you are quite<br>
welcome to do so). But I'd like to see this patch reviewed, improve<br>
and committed sooner rather than later because I'd like get this in<br>
before I start looking at cleaning up Zorg and documenting it.<br></blockquote><div><br></div><div>Glad to hear that you'll be giving zorg some love!</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
As a compromise we could mark the buildbot as experimental (so it<br>
doesn't notify devs like the normal buildbots) and have it only notify<br>
a few people (probably myself, you and anyone else who pipes up and<br>
says they're interested) about documentation build failures until we<br>
can reach a decision on whom it notifies</blockquote><div><br></div><div>That sounds fine. It looks like Tobias already has figured out how to do that so it shouldn't be difficult.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 (perhaps this issue should go<br>
to llvmdev as a separate thread?).<br></blockquote><div><br></div><div>Let's wait until we have some experience with the bot.</div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
Thanks,<br>
Dan.<br>
</blockquote></div><br></div></div>