<div class="gmail_quote">[From the correct email]</div><div class="gmail_quote"><br></div><div class="gmail_quote">On Mon, Apr 9, 2012 at 12:26 PM, Jarred Nicholls <span dir="ltr"><<a href="mailto:jarred@sencha.com">jarred@sencha.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><div>On Mon, Apr 9, 2012 at 12:18 PM, Ryosuke Niwa <span dir="ltr"><<a href="mailto:rniwa@webkit.org" target="_blank">rniwa@webkit.org</a>></span> wrote:</div>

</div><div class="gmail_quote"><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
We have discussed this before: <a href="https://lists.webkit.org/pipermail/webkit-dev/2011-September/017868.html" target="_blank">https://lists.webkit.org/pipermail/webkit-dev/2011-September/017868.html</a><div><br></div>


<div>A better change would be for us to generate the diff on EWS, and get rid of binding tests step from build bots since they aren't really testing anything. The primary use case of run-bindings-tests is to see the diff before/after a code generator change and it has no business running on the bot or in check-webkit-style.</div>


</blockquote><div><br></div></div><div>Either way, we concluded that running the binding tests is very quick and cheap, so wherever it is still effective to prevent regressions in code generation is fine by me; on an as-needed basis (EWS) is clearly ideal, rather than on all builds.</div>

<div class="im">
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span><font color="#888888">

<div><br></div></font></span><div><span><font color="#888888">- Ryosuke<br><br></font></span><div class="gmail_quote"><div><div>On Mon, Apr 9, 2012 at 6:39 AM, Vineet Chaudhary <span dir="ltr"><<a href="mailto:rgf748@motorola.com" target="_blank">rgf748@motorola.com</a>></span> wrote:<br>


</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>

<div><font face="arial, helvetica, sans-serif"><span style="border-collapse:collapse"><div style="margin-bottom:2em"><div><p style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px;font:normal normal normal 13px/normal Arial">






</p><p style="margin:0px 0px 0px 0px;font:13.0px Arial">Hi All,</p>
<p style="margin:0px 0px 0px 0px;font:13.0px Arial;min-height:15.0px"><br></p>
<p style="margin:0px 0px 0px 0px;font:13.0px Arial">It is observed that if changes are made in Codegenerator*.pm we need to rebase results of run-bindings-tests.</p>
<p style="margin:0px 0px 0px 0px;font:13.0px Arial">Many times authors forgot to update these binding results.</p><p style="margin:0px 0px 0px 0px;font:13.0px Arial"><br></p>
<p style="margin:0px 0px 0px 0px;font:13.0px Arial">IMO we should add check in ./check-webkit-style  to warn/(give a chance) author to run run-bindings-tests if Codegenerator is modified.</p>
<p style="margin:0px 0px 0px 0px;font:13.0px Arial">I have filed bug for this <a href="https://bugs.webkit.org/show_bug.cgi?id=83354" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=83354</a> .</p>
<p style="margin:0px 0px 0px 0px;font:13.0px Arial;min-height:15.0px"><br></p>
<p style="margin:0px 0px 0px 0px;font:13.0px Arial">Please let me know if any suggestions to make this change.</p><div><br></div><div>Thanks,</div><div>Vineet</div><p></p></div></div></span></font></div>
<br></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>
<br></div></blockquote></div><br></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>
<br></blockquote></div></div><span class="HOEnZb"><font color="#888888"><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>
</font></span></blockquote></div><br>