<div class="gmail_quote">On Wed, Sep 28, 2011 at 12:14 PM,  <span dir="ltr"><<a href="mailto:laszlo.1.gombos@nokia.com">laszlo.1.gombos@nokia.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

Hi,<br>
<div class="im"><br>
>>>>> If we don't support QT_NO_* officially and let the community fix<br>
>>>>> problems, we have to disable CONFIG+=qt_minimal on this bot,<br>
>>>>> because a core builder must be green almost at all times.<br>
>>>>><br>
>>>> IMO that's sensible. Do we provide guidance for people who want to<br>
>>>> maintain their own bots where they can test those configurations?<br>
>>><br>
>>> I agree. Having a core builder that builds a minimal _WebKit_ build<br>
>>> is something we should strive for, and enabling CONFIG+=qt_minimal<br>
>>> makes that bot more fragile and less useful for other ports<br>
>><br>
>> Seconded. The Qt Minimal bot should be minimal WebKit, and that's<br>
>> already on the edge of what we can reasonably expect non-Qt WebKit<br>
>> contributors to care about. I'm fine with keeping the QT_NO_* stuff in<br>
>> place if it's explicitly unsupported and doesn't grow into anything<br>
>> beyond disabled code blocks here and there.<br>
><br>
>Second on this, too.<br>
>If we keep the QT_NO* guards in tree we have to document somewhere that this is not officially supported. The minimal bot is otherwise an important one since it is the only one testing the correctness of at least a default set of the >gadzillion WebKit guards.<br>


<br>
</div>I would challenge this. I think the problem is that it is a really low bar to introduce a new build flag for trunk. We should keep the bar high for not only new "WebKit flags" but also for "Qt flags" on the trunk. Allowing a flag without a bot is allowing code without auto test. I do not think that having flags that almost build is any good.<br>


<br>
I would keep the minimal bot configuration as it is or create a new bot for CONFIG+=qt_minimal and enforce that every "Qt flags" is also being tested (so that at least it builds all the way).<br>
<br>
I also agree with Andreas that most QT_NO_* defines should go from WebKit trunk. We should start a review process and go trough each and every one of them one-by-one. <b>We should consider that some flags might be removed from Qt5 anyway.</b><br>

</blockquote><div><br></div><div>This is the reason I segued into Qt5 earlier.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
Laszlo<br>
<div><div></div><div class="h5">_______________________________________________<br>
webkit-qt mailing list<br>
<a href="mailto:webkit-qt@lists.webkit.org">webkit-qt@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-qt" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-qt</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><span style="color:rgb(214, 214, 214);font-family:Helvetica;font-size:medium"><div style="color:rgb(0, 0, 0)"><font color="#D6D6D6">................................................................</font></div>

<div style="color:rgb(0, 0, 0)"><br></div><div style="color:rgb(0, 0, 0)"><b><font color="#2D770A">Sencha</font></b></div><div style="color:rgb(0, 0, 0)">Jarred Nicholls<font color="#5F5E5F">, </font><font color="#929292">Senior Software Architect</font></div>

<div><a href="http://twitter.com/jarrednicholls" style="color:rgb(0, 0, 204)" target="_blank"><font color="#003F72">@jarrednicholls<br></font></a></div></span><br>