Bug 19311

Summary:

On Mac, no way to differentiate Apple port from others in build scripts

Product:

WebKit

Reporter:

Kevin Ollivier <kevino>

Component:

WebKit Misc.

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

Keywords:

Gtk, Qt, Wx

Priority:

Version:

528+ (Nightly build)

Hardware:

Mac

OS:

OS X 10.5

Attachments:

Description	Flags
Restore original behavior of isOSX()	darin: review+

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! :)