Summary: | Clean up imports for webkit-patch. | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | James Kozianski <koz> | ||||||||||||
Component: | New Bugs | Assignee: | James Kozianski <koz> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | abarth, commit-queue, eric, noel.gordon, ojan | ||||||||||||
Priority: | P2 | ||||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||
Hardware: | Other | ||||||||||||||
OS: | OS X 10.5 | ||||||||||||||
Attachments: |
|
Description
James Kozianski
2010-11-05 11:47:21 PDT
Created attachment 73099 [details]
Patch
Comment on attachment 73099 [details]
Patch
Good clean up. Please sort the commands (as per unix sort command).
Created attachment 73112 [details]
Patch
Comment on attachment 73112 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=73112&action=review > WebKitTools/Scripts/webkitpy/tool/main.py:46 > +from optparse import make_option > +from webkitpy.common.net.statusserver import StatusServer Can these not be in sorted order? I realize the imports surrounding it aren't, but they really should be if they can. Created attachment 73120 [details]
Patch
Comment on attachment 73099 [details]
Patch
This was breaking teh win-ews.
This patch didn't seem to apply on any of the EWS bots. I assume it won't on the commit-queue either. Any ideas why? Created attachment 73294 [details]
Patch
Sorry, my bad - my checkout was out of date. Comment on attachment 73294 [details] Patch Rejecting patch 73294 from commit-queue. Failed to run "['./WebKitTools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=abarth-cq-sl', 'apply-attachment', '--force-clean', '--non-interactive', 73294]" exit_code: 1 Traceback (most recent call last): File "./WebKitTools/Scripts/webkit-patch", line 70, in <module> main() File "./WebKitTools/Scripts/webkit-patch", line 63, in main from webkitpy.tool.main import WebKitPatch File "/Users/abarth/git/webkit-queue/WebKitTools/Scripts/webkitpy/tool/main.py", line 42, in <module> from webkitpy.common.net.rietveld import Rietveld ImportError: No module named rietveld Full output: http://queues.webkit.org/results/5577007 Created attachment 73395 [details]
Patch
Apologies for the failure - it was running fine on my box because there were still lingering .pyc files for .py files that were removed in prior patches. Comment on attachment 73395 [details]
Patch
Looks fine, assuming it runs. Last time this patch broke the win-ews :(
Comment on attachment 73395 [details] Patch Clearing flags on attachment: 73395 Committed r71667: <http://trac.webkit.org/changeset/71667> All reviewed patches have been landed. Closing bug. |