<meta charset="utf-8"><span class="Apple-style-span" style="font-family: arial, sans-serif; font-size: 13px; border-collapse: collapse; "><div>This approach doesn&#39;t lend itself as well to trying patches before putting them up for review. Specifically, I want to be able to  try patches without spamming everyone with bugzilla mail. This is solvable in this bugzilla-based approach, but it doesn&#39;t lend itself to this as naturally, e.g. presumably there&#39;s a way to tell bugzilla not to send mail for a given comment.</div>

<div><br></div><div>Also, it would be great if the commit-queue, try-server, whatever, had a UI like the buildbot waterfall. There&#39;s a couple advantages:</div><div>1. Can see the stdio as the tests run and get better information about why it failed. </div>

<div>2. Can grab layout test results from the try servers. This would reduce the need/occurence of committing Mac expectations and then cleaning up other platforms post commit.</div><div><br></div><font color="#888888"><div>

<font class="Apple-style-span" color="#000000">Ojan</font></div></font></span><br><div class="gmail_quote">On Thu, Nov 12, 2009 at 12:58 PM, Brian Weinstein <span dir="ltr">&lt;<a href="mailto:bweinstein@apple.com">bweinstein@apple.com</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div style="word-wrap:break-word">Seconded (or Thirded). I&#39;d been working on a try-server using Chromium&#39;s try-change.py, but this seems like a much cleaner way to handle it, and ties into the Bugzilla workflow much better than my solution, and would be much easier to limit who can set the try bit, based on what we decide the policy to be.<div>

<div></div><div class="h5"><div> <br><div><div>On Nov 12, 2009, at 12:41 PM, Jeremy Orlow wrote:</div><br><blockquote type="cite">It&#39;s so easy to have code that builds on one platform but not another.  Even if the try servers were only builders to begin with, I think they&#39;d provide a lot of value to the project.<br>

<br><div class="gmail_quote">

On Thu, Nov 12, 2009 at 11:43 AM, Kenneth Christiansen <span dir="ltr">&lt;<a href="mailto:kenneth.christiansen@openbossa.org" target="_blank">kenneth.christiansen@openbossa.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



I think that sounds like a really good idea, and I can see my self<br>
using that when touching cross platform code.<br>
<br>
Kenneth<br>
<div><div></div><div><br>
On Thu, Nov 12, 2009 at 4:37 PM, Adam Barth &lt;<a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a>&gt; wrote:<br>
&gt; As the project grows, we need to scale our processes to match.  In<br>
&gt; large part, that means automating as much work as possible.<br>
&gt; Commit-queue has done a good job of solving the &quot;land patches from<br>
&gt; non-committers efficiently&quot; problem, effectively removing that as a<br>
&gt; pain point.  I&#39;d like to ask you to open your hearts and your minds to<br>
&gt; the idea of automating more of our processes.<br>
&gt;<br>
&gt; Currently, I see the biggest pain-point in our process as the<br>
&gt; always-burgeoning pending-review list.  It&#39;s difficult to automate the<br>
&gt; process of accepting good patches because that requires attention from<br>
&gt; experts.  Instead, I think we should make it easier to reject bad<br>
&gt; patches.  As a first step, I&#39;ve started extending bugzilla-tool to be<br>
&gt; a try server in &lt;<a href="https://bugs.webkit.org/show_bug.cgi?id=31422" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=31422</a>&gt;.<br>
&gt; Here&#39;s how this might work:<br>
&gt;<br>
&gt; 1) Contributor posts patch for review.<br>
&gt; 2) Committer marks patch with the try? flag.<br>
&gt; 3) The try-queue downloads, applies, builds, and tests the patch.<br>
&gt; 4) If all systems are go, the try-queue marks the patch as try+.<br>
&gt; Otherwise, it marks the patch as try- with an explanation of what went<br>
&gt; wrong.<br>
&gt;<br>
&gt; The try-queue will be purely optional and advisory.  Hopefully a try-<br>
&gt; notation will encourage the contributor to post a new version of the<br>
&gt; patch that passes the try-queue.<br>
&gt;<br>
&gt; Further down the road, one can also imagine another bot that automates<br>
&gt; step (2) by scanning the pending-review list for untried patches and<br>
&gt; marking them as try? when the try-queue has unused bandwidth.<br>
&gt;<br>
&gt; Adam<br>
&gt; _______________________________________________<br>
&gt; webkit-dev mailing list<br>
&gt; <a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
&gt; <a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
&gt;<br>
<br>
<br>
<br>
</div></div><font color="#888888">--<br>
Kenneth Rohde Christiansen<br>
Technical Lead / Software Engineer<br>
Qt Labs Americas, Nokia Technology Institute, INdT<br>
Phone  +55 81 8895 6002 / E-mail kenneth.christiansen at <a href="http://openbossa.org/" target="_blank">openbossa.org</a><br>
</font><div><div></div><div>_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
</div></div></blockquote></div><br>
_______________________________________________<br>webkit-dev mailing list<br><a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br><a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>

</blockquote></div><br></div></div></div></div><br>_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
<br></blockquote></div><br>