new-run-webkit-httpd imports common/host.py which imports lots of stuff including common/net/buildbot.py, which will fail to import the json module.<div><br></div><div>I would land smaller pieces so the rollback isn't as painful. :)<br>
<div><br><div class="gmail_quote">On Thu, Nov 17, 2011 at 4:42 PM, Adam Barth <span dir="ltr"><<a href="mailto:abarth@webkit.org">abarth@webkit.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
The remaining things seem related to websocket-server and<br>
new-run-webkit-httpd, which I don't think will be effected by your<br>
change.<br>
<span class="HOEnZb"><font color="#888888"><br>
Adam<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On Thu, Nov 17, 2011 at 4:40 PM, Eric Seidel <<a href="mailto:eric@webkit.org">eric@webkit.org</a>> wrote:<br>
> Other bot steps in Chromium depend on webkitpy?<br>
><br>
> I'm tempted to just make the change and see what breaks.  We can<br>
> always roll it out if things are really bad.<br>
><br>
> I'll prepare an updated patch.<br>
><br>
> -eric<br>
><br>
> On Thu, Nov 17, 2011 at 4:37 PM, Tony Chang <<a href="mailto:tony@chromium.org">tony@chromium.org</a>> wrote:<br>
>> Only new-run-webkit-tests uses python 2.7 on the leopard bots.  There are<br>
>> other bot steps in chromium that would break.  I mention a couple cases<br>
>> here:<br>
>> <a href="http://code.google.com/p/chromium/issues/detail?id=103266#c6" target="_blank">http://code.google.com/p/chromium/issues/detail?id=103266#c6</a><br>
>> Alternately, we could try to fully switch the leopard bots to 2.7<br>
>> (<a href="http://crbug.com/103828" target="_blank">http://crbug.com/103828</a>), but that's a larger task.<br>
>><br>
>> On Thu, Nov 17, 2011 at 3:19 PM, Dirk Pranke <<a href="mailto:dpranke@chromium.org">dpranke@chromium.org</a>> wrote:<br>
>>><br>
>>> On Thu, Nov 17, 2011 at 2:45 PM, Adam Barth <<a href="mailto:abarth@webkit.org">abarth@webkit.org</a>> wrote:<br>
>>> > On Thu, Nov 17, 2011 at 2:42 PM, Dirk Pranke <<a href="mailto:dpranke@chromium.org">dpranke@chromium.org</a>><br>
>>> > wrote:<br>
>>> >> The Chromium Leopard bots are still using 2.5 as far as I know. Unless<br>
>>> >> "move forward" includes you upgrading those bots, you shouldn't remove<br>
>>> >> the 2.5 compat code until they have been upgraded. (If you are signing<br>
>>> >> up to upgrade them, then great!).<br>
>>> ><br>
>>> > Tony already upgraded the bots.<br>
>>> ><br>
>>> > Adam<br>
>>> ><br>
>>><br>
>>> I have been corrected. You have my blessings :).<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>
>><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>
>><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>
</div></div></blockquote></div><br></div></div>