Bug 31861 - Update pywebsocket to 0.4.2
Summary: Update pywebsocket to 0.4.2
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: 31604
  Show dependency treegraph
 
Reported: 2009-11-25 01:05 PST by Yuzo Fujishima
Modified: 2009-11-25 22:16 PST (History)
2 users (show)

See Also:


Attachments
Update pywebsocket to 0.4.2 (20.26 KB, patch)
2009-11-25 01:07 PST, Yuzo Fujishima
no flags Details | Formatted Diff | Diff
Update pywebsocket to 0.4.2 (17.34 KB, patch)
2009-11-25 18:04 PST, Yuzo Fujishima
no flags Details | Formatted Diff | Diff
Update pywebsocket to 0.4.2 (16.43 KB, patch)
2009-11-25 18:05 PST, Yuzo Fujishima
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yuzo Fujishima 2009-11-25 01:05:33 PST
Update pywebsocket to 0.4.2 (http://code.google.com/p/pywebsocket/).

In the new version a few bugs have been fixed, including one for https://bugs.webkit.org/show_bug.cgi?id=31603 (already fixed in the WebKit copy).
Also, the new version has better logging, which is needed for https://bugs.webkit.org/show_bug.cgi?id=31604
Comment 1 Yuzo Fujishima 2009-11-25 01:07:20 PST
Created attachment 43830 [details]
Update pywebsocket to 0.4.2
Comment 2 Eric Seidel (no email) 2009-11-25 07:49:18 PST
Comment on attachment 43830 [details]
Update pywebsocket to 0.4.2

LGTM too, but it look slike the new files are missing.  Was this uploaded by 'bugzilla-tool post-diff' or post-commits?  Those should both use --binary and correctly include new files.
Comment 3 Yuzo Fujishima 2009-11-25 18:04:57 PST
Created attachment 43883 [details]
Update pywebsocket to 0.4.2
Comment 4 Yuzo Fujishima 2009-11-25 18:05:47 PST
Created attachment 43884 [details]
Update pywebsocket to 0.4.2
Comment 5 Yuzo Fujishima 2009-11-25 18:11:19 PST
Hi, thank you for the review.

The file permission for the .py files were 755 for the first patch.
That might have caused the commit issue?

I've changed the permission to 644, which is the same as that of the files
already in the repository.

(I used bugzilla-tool  post-commits.
bugzilla-tool help post-commits/post-diff doesn't show --binary
option and bugzilla-tool actually rejected the option for me. So I haven't
used it.)

Yuzo
Comment 6 Eric Seidel (no email) 2009-11-25 22:04:03 PST
Comment on attachment 43884 [details]
Update pywebsocket to 0.4.2

I think I was simply confused.  I'm not sure there was any issue w/ the first patch.
Comment 7 WebKit Commit Bot 2009-11-25 22:16:18 PST
Comment on attachment 43884 [details]
Update pywebsocket to 0.4.2

Clearing flags on attachment: 43884

Committed r51406: <http://trac.webkit.org/changeset/51406>
Comment 8 WebKit Commit Bot 2009-11-25 22:16:24 PST
All reviewed patches have been landed.  Closing bug.