Bug 19311 - On Mac, no way to differentiate Apple port from others in build scripts
Summary: On Mac, no way to differentiate Apple port from others in build scripts
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords: Gtk, Qt, Wx
Depends on:
Blocks:
 
Reported: 2008-05-29 10:46 PDT by Kevin Ollivier
Modified: 2008-05-29 11:04 PDT (History)
0 users

See Also:


Attachments
Restore original behavior of isOSX() (1.53 KB, patch)
2008-05-29 10:47 PDT, Kevin Ollivier
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kevin Ollivier 2008-05-29 10:46:31 PDT
In r33446, a change was made to webkitdirs.pm that removed the isDarwin() check and changed isOSX() to behave like isDarwin(). However, isOSX() has always been used for code intended only for the Apple port, not other ports running on OS X, so now ports using build-webkit get the properties/settings intended for the Apple port.
Comment 1 Kevin Ollivier 2008-05-29 10:47:16 PDT
Created attachment 21415 [details]
Restore original behavior of isOSX()
Comment 2 Darin Adler 2008-05-29 10:57:47 PDT
Comment on attachment 21415 [details]
Restore original behavior of isOSX()

Seems fine.

Although I would probably use a different phrase from "Apple port".

Please don't check in the double-ChangeLog entry.
Comment 3 Kevin Ollivier 2008-05-29 11:04:11 PDT
landed in r34206, thanks! :)