Bug 33036

Summary: [bzt] post-diff requires reading to the end
Product: WebKit Reporter: Adam Barth <abarth>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, eric, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

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.