Bug 91682 - config_standalone.py is crashing on apple win's cygwin port
Summary: config_standalone.py is crashing on apple win's cygwin port
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:
Depends on:
Blocks:
 
Reported: 2012-07-18 15:28 PDT by Dirk Pranke
Modified: 2012-07-19 12:14 PDT (History)
6 users (show)

See Also:


Attachments
Patch (1.89 KB, patch)
2012-07-18 15:33 PDT, Dirk Pranke
no flags 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-07-18 15:28:05 PDT
this seems to be due to the top-level import of ctypes in webkitpy.common.system.executive.py, which is only needed on win32. I don't know what ctypes is doing to cause the problem, or why it is only happening in config_standalone.
Comment 1 Dirk Pranke 2012-07-18 15:33:50 PDT
Created attachment 153108 [details]
Patch
Comment 2 Dirk Pranke 2012-07-18 15:35:12 PDT
to reproduce you can run 'python config_standalone.py --mock Debug' from the cygwin command prompt, and you should see "Debug\nAborted" with python crashing at process exit :(.

You of course may not be able to reproduce this, as it doesn't seem to happen on the bots.
Comment 3 Adam Roben (:aroben) 2012-07-19 06:11:40 PDT
Comment on attachment 153108 [details]
Patch

Too bad we can't use the win32api module instead. Ah, the joys of Cygwin!
Comment 4 WebKit Review Bot 2012-07-19 12:14:06 PDT
Comment on attachment 153108 [details]
Patch

Clearing flags on attachment: 153108

Committed r123133: <http://trac.webkit.org/changeset/123133>
Comment 5 WebKit Review Bot 2012-07-19 12:14:11 PDT
All reviewed patches have been landed.  Closing bug.