Bug 33036 - [bzt] post-diff requires reading to the end
Summary: [bzt] post-diff requires reading to the end
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-12-29 18:29 PST by Adam Barth
Modified: 2009-12-29 20:12 PST (History)
3 users (show)

See Also:


Attachments
Patch (1.30 KB, patch)
2009-12-29 18:39 PST, Adam Barth
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Barth 2009-12-29 18:29:05 PST
If you don't read to the end of the confirm screen, you get this error:

Traceback (most recent call last):
  File "./WebKitTools/Scripts/bugzilla-tool", line 108, in <module>
    BugzillaTool().main()
  File "/Users/abarth/svn/kr/src/third_party/WebKit/WebKitTools/Scripts/modules/multicommandtool.py", line 283, in main
    return command.check_arguments_and_execute(options, args, self)
  File "/Users/abarth/svn/kr/src/third_party/WebKit/WebKitTools/Scripts/modules/multicommandtool.py", line 111, in check_arguments_and_execute
    return self.execute(options, args, tool) or 0
  File "/Users/abarth/svn/kr/src/third_party/WebKit/WebKitTools/Scripts/modules/commands/download.py", line 66, in execute
    self._sequence.run_and_handle_errors(tool, options, self._prepare_state(options, args, tool))
  File "/Users/abarth/svn/kr/src/third_party/WebKit/WebKitTools/Scripts/modules/stepsequence.py", line 65, in run_and_handle_errors
    self._run(tool, options, state)
  File "/Users/abarth/svn/kr/src/third_party/WebKit/WebKitTools/Scripts/modules/stepsequence.py", line 59, in _run
    step(tool, options).run(state)
  File "/Users/abarth/svn/kr/src/third_party/WebKit/WebKitTools/Scripts/modules/buildsteps.py", line 234, in run
    self._tool.user.page(diff)
  File "/Users/abarth/svn/kr/src/third_party/WebKit/WebKitTools/Scripts/modules/user.py", line 43, in page
    child_process.communicate(input=message)
  File "/System/Library/Frameworks/Python.framework/Versions/2.6/lib/python2.6/subprocess.py", line 660, in communicate
    self.stdin.write(input)
IOError: [Errno 32] Broken pipe
Comment 1 Adam Barth 2009-12-29 18:39:39 PST
Created attachment 45628 [details]
Patch
Comment 2 WebKit Review Bot 2009-12-29 18:44:46 PST
style-queue ran check-webkit-style on attachment 45628 [details] without any errors.
Comment 3 Eric Seidel (no email) 2009-12-29 20:02:36 PST
Comment on attachment 45628 [details]
Patch

OK.
Comment 4 WebKit Commit Bot 2009-12-29 20:12:17 PST
Comment on attachment 45628 [details]
Patch

Clearing flags on attachment: 45628

Committed r52639: <http://trac.webkit.org/changeset/52639>
Comment 5 WebKit Commit Bot 2009-12-29 20:12:27 PST
All reviewed patches have been landed.  Closing bug.