Bug 78866 - webkit-patch is broken on SL with a new, clean checkout, using Python 2.6.1
Summary: webkit-patch is broken on SL with a new, clean checkout, using Python 2.6.1
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dirk Pranke
URL:
Keywords:
: 78931 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-02-16 18:53 PST by Dirk Pranke
Modified: 2012-02-17 13:53 PST (History)
5 users (show)

See Also:


Attachments
Patch (3.27 KB, patch)
2012-02-17 13:50 PST, Dirk Pranke
abarth: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Pranke 2012-02-16 18:53:18 PST
On my corp Mac Pro (running SL + Python 2.6.1), if I have a clean checkout (or remove my webkitpy.thirdparty.autoinstalled directory, then "webkit-patch help" fails trying to auto install irclib :(
Comment 1 Dirk Pranke 2012-02-16 18:54:56 PST
it works fine, however, on Lion with python 2.7.1 .
Comment 2 Dirk Pranke 2012-02-16 18:57:45 PST
I've confirmed this on two different machines running the same corp image, so I don't think it's just one machine. I will see if maybe it has something to do with this specific version of irc.

The script keels over trying to decompress the zipfile into a temporary directory; for some reason it is creating the top-level directory python-irclib-0.4.8 as a file, not a directory, and then everything goes south.


Traceback (most recent call last):
  File "/src/wk/Tools/Scripts/webkit-patch", line 44, in <module>
    from webkitpy.tool.main import WebKitPatch
  File "/Volumes/Source/src/wk/Tools/Scripts/webkitpy/tool/main.py", line 38, in <module>
    from webkitpy.common.net.irc import ircproxy
  File "/Volumes/Source/src/wk/Tools/Scripts/webkitpy/common/net/irc/ircproxy.py", line 31, in <module>
    from webkitpy.common.net.irc.ircbot import IRCBot
  File "/Volumes/Source/src/wk/Tools/Scripts/webkitpy/common/net/irc/ircbot.py", line 32, in <module>
    from webkitpy.thirdparty.autoinstalled.irc import ircbot
  File "/Volumes/Source/src/wk/Tools/Scripts/webkitpy/thirdparty/__init__.py", line 80, in find_module
    self._install_irc()
  File "/Volumes/Source/src/wk/Tools/Scripts/webkitpy/thirdparty/__init__.py", line 123, in _install_irc
    url_subpath="irclib.py")
  File "/Volumes/Source/src/wk/Tools/Scripts/webkitpy/common/system/autoinstall.py", line 445, in install
    raise Exception(message)
Exception: Error auto-installing the irclib.py package to:
 "/Volumes/Source/src/wk/Tools/Scripts/webkitpy/thirdparty/autoinstalled/irc/irclib.py"
 --> Inner message: [Errno 20] Not a directory: '/var/folders/++/++3pXE++6+0++4RjPqRgNE+-RMo/-Tmp-/irclib.py_YZR9KZ/python-irclib-0.4.8/dccsend'
Comment 3 Eric Seidel (no email) 2012-02-16 18:58:16 PST
I suspect it failed to download the package from Adam's server. :)
Comment 4 Dirk Pranke 2012-02-16 19:08:04 PST
(In reply to comment #3)
> I suspect it failed to download the package from Adam's server. :)

Nope, package is downloading fine. I actually changed the version to 0.4.6 to try that, and it failed w/ the same error.
Comment 5 Adam Barth 2012-02-17 10:32:33 PST
Can you try reverting the recent change to the autoinstaller to remove 2.5 compat?
Comment 6 Adam Barth 2012-02-17 10:32:56 PST
"to remove" => "that removed"
Comment 7 Dirk Pranke 2012-02-17 13:37:21 PST
(In reply to comment #5)
> Can you try reverting the recent change to the autoinstaller to remove 2.5 compat?

Oh, you mean, can I revert the change where we stopped using our working code and decided to use the broken system version? Yeah, that fixes the problem :).

I've reopened bug 78173.
Comment 8 Adam Barth 2012-02-17 13:49:23 PST
> > Can you try reverting the recent change to the autoinstaller to remove 2.5 compat?
> 
> Oh, you mean, can I revert the change where we stopped using our working code and decided to use the broken system version? Yeah, that fixes the problem :).

Yes.  :)
Comment 9 Dirk Pranke 2012-02-17 13:50:23 PST
*** Bug 78931 has been marked as a duplicate of this bug. ***
Comment 10 Dirk Pranke 2012-02-17 13:50:50 PST
Created attachment 127648 [details]
Patch
Comment 11 Adam Barth 2012-02-17 13:52:10 PST
Comment on attachment 127648 [details]
Patch

Thanks for updating the comment.
Comment 12 Eric Seidel (no email) 2012-02-17 13:53:49 PST
What an excellent example of why good "why" comments are important. :)

Thanks Dirk!
Comment 13 Dirk Pranke 2012-02-17 13:53:54 PST
Committed r108118: <http://trac.webkit.org/changeset/108118>