Bug 136486

Summary: [Win] Revise test environment for Windows build
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: Tools / TestsAssignee: Brent Fulgham <bfulgham>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, dbates, ddkilzer
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Attachments:
Description Flags
Patch ddkilzer: review+

Description Brent Fulgham 2014-09-03 10:37:23 PDT 
The Windows build and test system has some cruft that needs to be cleaned up:
1. DRT and friends know how to find 64-bit runtimes, so the special cases in webkitdirs.pm should be removed.
2. We don't need to look for Safari.exe on Windows to run tests, and shouldn't complain if we don't find it.
3. Where possible, we should use File::Spec for file handling so that we can be agnostic about whether the tests are being run under Cygwin or not.
Comment 1 Brent Fulgham 2014-09-03 11:09:38 PDT 
Created attachment 237562 [details]
Patch
Comment 2 Brent Fulgham 2014-09-03 11:15:37 PDT 
*** Bug 125180 has been marked as a duplicate of this bug. ***
Comment 3 Brent Fulgham 2014-09-03 13:01:15 PDT 
I didn't actually make any File::Spec changes here. That will be coming in a separate change.
Comment 4 David Kilzer (:ddkilzer) 2014-09-03 13:05:58 PDT 
Comment on attachment 237562 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=237562&action=review

r=me

> Tools/Scripts/webkitdirs.pm:765
> +    die "Safari Path is only relevant on Mac platform\n" unless isAppleMacWebKit();

Nit:  Lowercase "Path"?  Maybe say instead:

    die "Safari path is only relevant on Apple Mac platform\n" unless isAppleMacWebKit();

> Tools/Scripts/webkitdirs.pm:784
> +    die "Safari Path is only relevant on Mac platform\n" unless isAppleMacWebKit();

Same:

    die "Safari path is only relevant on Apple Mac platform\n" unless isAppleMacWebKit();
Comment 5 Brent Fulgham 2014-09-03 13:12:43 PDT 
Committed r173219: <http://trac.webkit.org/changeset/173219>