Fixing the problem will likely take less time at this point than rolling the patch out as there are a number of dependent patches. The fix should be in soon.<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 20, 2012 at 11:10 AM, Geoffrey Garen <span dir="ltr">&lt;<a href="mailto:ggaren@apple.com" target="_blank">ggaren@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">I&#39;d prefer to see the patch rolled out.<br>
<br>
Geoff<br>
<div class="HOEnZb"><div class="h5"><br>
On Sep 20, 2012, at 11:07 AM, Alexey Proskuryakov &lt;<a href="mailto:ap@webkit.org">ap@webkit.org</a>&gt; wrote:<br>
<br>
&gt;<br>
&gt; Now tracked as &lt;<a href="https://bugs.webkit.org/show_bug.cgi?id=97182" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=97182</a>&gt;.<br>
&gt;<br>
&gt; I think that we should close the tree if resolving this takes any significant time. Not being able to see how exactly tests are failing on other platforms is unacceptable.<br>
&gt;<br>
&gt; - WBR, Alexey Proskuryakov<br>
&gt;<br>
&gt;<br>
&gt; 20.09.2012, в 3:54, Osztrogonac Csaba &lt;<a href="mailto:oszi@inf.u-szeged.hu">oszi@inf.u-szeged.hu</a>&gt; написал(а):<br>
&gt;<br>
&gt;&gt; Unfortunately r129047 broke the results.html, see <a href="https://bugs.webkit.org/show_bug.cgi?id=96845#c9" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=96845#c9</a> for details.<br>
&gt;&gt;<br>
&gt;&gt; Dirk Pranke írta:<br>
&gt;&gt;&gt; These changes are now starting to land ...<br>
&gt;&gt;&gt; as of r129047, TEXT, IMAGE+TEXT, and AUDIO are no longer legal<br>
&gt;&gt;&gt; keywords in the TestExpectations syntax ... you should use &quot;FAIL&quot;<br>
&gt;&gt;&gt; instead.<br>
&gt;&gt;&gt; I will be landing the support for the new syntax as quickly as I can<br>
&gt;&gt;&gt; to minimize the transition period. Apologies for the inconvenience.<br>
&gt;&gt;&gt; -- Dirk<br>
&gt;&gt;&gt; On Wed, Sep 12, 2012 at 4:29 PM, Dirk Pranke &lt;<a href="mailto:dpranke@chromium.org">dpranke@chromium.org</a>&gt; wrote:<br>
&gt;&gt;&gt;&gt; Hi all,<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; The new format of the much-debated TestExpectations syntax will be<br>
&gt;&gt;&gt;&gt; landing soon (hopefully in the next couple days).<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; For those of who have forgotten / repressed the earlier debates, the<br>
&gt;&gt;&gt;&gt; new syntax looks something like:<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; <a href="http://webkit.org/b/12345" target="_blank">webkit.org/b/12345</a> [ Mac Vista] fast/html/keygen.html [ ImageOnlyFailure ]<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Andis documented in full at<br>
&gt;&gt;&gt;&gt; <a href="https://trac.webkit.org/wiki/TestExpectations#NewSyntaxNotquiteyetlanded" target="_blank">https://trac.webkit.org/wiki/TestExpectations#NewSyntaxNotquiteyetlanded</a><br>
&gt;&gt;&gt;&gt; .<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; ( The &quot;[&quot; and &quot;]&quot; characters are delimiters, not EBNF &quot;optional&quot;<br>
&gt;&gt;&gt;&gt; markers, although those sections are in fact optional :) ).<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Note that the new syntax means that Skipped files are a syntactic<br>
&gt;&gt;&gt;&gt; subset of TestExpectations files, and I plan to convert all of the<br>
&gt;&gt;&gt;&gt; Skipped files to TestExpectations files via copy and paste shortly<br>
&gt;&gt;&gt;&gt; after the new syntax is landed, and then drop support for Skipped<br>
&gt;&gt;&gt;&gt; files (I will update ORWT to use the new files and treat any entry as<br>
&gt;&gt;&gt;&gt; a Skip).<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; The plan for landing these changes is:<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; 1) Add support for parsing the new lines and converting them back into<br>
&gt;&gt;&gt;&gt; the old format (internally) so that both syntaxes are supported<br>
&gt;&gt;&gt;&gt; 2) Convert all the existing files over<br>
&gt;&gt;&gt;&gt; 3) Make sure things aren&#39;t broken :)<br>
&gt;&gt;&gt;&gt; 4) Drop support for the old syntax<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; I plan for this to all happen quickly, in less than a day. This means<br>
&gt;&gt;&gt;&gt; that if you have patches posted that modify those files they may<br>
&gt;&gt;&gt;&gt; become stale and need to be updated.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Changes from the old syntax:<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; 1. We use URLs (a specific whitelisted set; let me know if you want to<br>
&gt;&gt;&gt;&gt; add to it) instead of BUGWK12345 etc.<br>
&gt;&gt;&gt;&gt; 2. We use &quot;bug(dpranke) instead of BUGDPRANKE<br>
&gt;&gt;&gt;&gt; 3. We use CamelCase instead of SHOUTING<br>
&gt;&gt;&gt;&gt; 4. We use Failure to represent what used to be TEXT, IMAGE+TEXT, and<br>
&gt;&gt;&gt;&gt; AUDIO - these failures will be indistinguishable in the new world,<br>
&gt;&gt;&gt;&gt; meaning that you can&#39;t distinguish between &quot;text only&quot; and &quot;both image<br>
&gt;&gt;&gt;&gt; and text&quot;. Since only Chromium runs pixel tests by default, this<br>
&gt;&gt;&gt;&gt; shouldn&#39;t be a big deal.<br>
&gt;&gt;&gt;&gt; 5. We use ImageOnlyFailure to represent what used to be IMAGE<br>
&gt;&gt;&gt;&gt; 6. We use &quot;[&quot; and &quot;]&quot; for delimiters instead of &quot;:&quot; and &quot;=&quot;<br>
&gt;&gt;&gt;&gt; 7. We use &quot;#&quot; instead of &quot;//&quot; as a comment<br>
&gt;&gt;&gt;&gt; 7. WontFix will now imply Skip, i.e., tests marked WontFix will<br>
&gt;&gt;&gt;&gt; automatically be Skipped<br>
&gt;&gt;&gt;&gt; 8. WontFix and Skip will not require (or even allow) any other<br>
&gt;&gt;&gt;&gt; expectations, i.e., you can&#39;t say &quot;[ WontFix Crash ]&quot;. If you want to<br>
&gt;&gt;&gt;&gt; indicate that the test will crash if you actually run it, use a<br>
&gt;&gt;&gt;&gt; comment.<br>
&gt;&gt;&gt;&gt; 9. WontFix, Skip, Slow, and Rebaseline all move from the left hand<br>
&gt;&gt;&gt;&gt; side to the right. The only keywords on the left restrict which<br>
&gt;&gt;&gt;&gt; configurations the lines apply to.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; I will send out follow-up emails as this stuff lands. Please let me<br>
&gt;&gt;&gt;&gt; know if you have any questions. Thanks!<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; -- Dirk<br>
&gt;&gt;&gt; _______________________________________________<br>
&gt;&gt;&gt; webkit-dev mailing list<br>
&gt;&gt;&gt; <a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
&gt;&gt;&gt; <a href="http://lists.webkit.org/mailman/listinfo/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
&gt;&gt;<br>
&gt;&gt; _______________________________________________<br>
&gt;&gt; webkit-dev mailing list<br>
&gt;&gt; <a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
&gt;&gt; <a href="http://lists.webkit.org/mailman/listinfo/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; - WBR, Alexey Proskuryakov<br>
&gt;<br>
&gt;<br>
&gt; _______________________________________________<br>
&gt; webkit-dev mailing list<br>
&gt; <a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
&gt; <a href="http://lists.webkit.org/mailman/listinfo/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
<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/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
</div></div></blockquote></div><br></div>