Bug 151721

Summary: [Win] Run non-http tests without Cygwin
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: Tools / TestsAssignee: Brent Fulgham <bfulgham>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, glenn, lforschler
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch aestes: review+

Description Brent Fulgham 2015-12-01 14:13:21 PST
The attached patch allows you to run the WebKit LayoutTest suite without needing Cygwin. It does not yet support HTTP tests.
Comment 1 Brent Fulgham 2015-12-01 16:39:15 PST
Created attachment 266409 [details]
Patch
Comment 2 WebKit Commit Bot 2015-12-01 16:42:06 PST
Attachment 266409 [details] did not pass style-queue:


ERROR: Tools/Scripts/webkitpy/port/win.py:231:  [WinPort.read_registry_value] Undefined variable 'WindowsError'  [pylint/E0602] [5]
ERROR: Tools/Scripts/webkitpy/port/win.py:257:  [WinPort.write_registry_value] Undefined variable 'WindowsError'  [pylint/E0602] [5]
ERROR: Tools/Scripts/webkitpy/port/win.py:261:  [WinPort.write_registry_value] Undefined variable 'WindowsError'  [pylint/E0602] [5]
Total errors found: 3 in 24 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Brent Fulgham 2015-12-01 16:42:52 PST
(In reply to comment #2)
> Attachment 266409 [details] did not pass style-queue:
> 
> 
> ERROR: Tools/Scripts/webkitpy/port/win.py:231: 
> [WinPort.read_registry_value] Undefined variable 'WindowsError' 
> [pylint/E0602] [5]
> ERROR: Tools/Scripts/webkitpy/port/win.py:257: 
> [WinPort.write_registry_value] Undefined variable 'WindowsError' 
> [pylint/E0602] [5]
> ERROR: Tools/Scripts/webkitpy/port/win.py:261: 
> [WinPort.write_registry_value] Undefined variable 'WindowsError' 
> [pylint/E0602] [5]
> Total errors found: 3 in 24 files
> 
> 
> If any of these errors are false positives, please file a bug against
> check-webkit-style.

These warnings are false positives. WindowsError is present in the native Windows Python implementation.
Comment 4 Andy Estes 2015-12-01 18:15:01 PST
Comment on attachment 266409 [details]
Patch

rs=me
Comment 5 Brent Fulgham 2015-12-02 08:53:23 PST
Committed r192944: <http://trac.webkit.org/changeset/192944>