<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>19311</bug_id>
          
          <creation_ts>2008-05-29 10:46:31 -0700</creation_ts>
          <short_desc>On Mac, no way to differentiate Apple port from others in build scripts</short_desc>
          <delta_ts>2008-05-29 11:04:11 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit Misc.</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Gtk, Qt, Wx</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Kevin Ollivier">kevino</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>81864</commentid>
    <comment_count>0</comment_count>
    <who name="Kevin Ollivier">kevino</who>
    <bug_when>2008-05-29 10:46:31 -0700</bug_when>
    <thetext>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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>81865</commentid>
    <comment_count>1</comment_count>
      <attachid>21415</attachid>
    <who name="Kevin Ollivier">kevino</who>
    <bug_when>2008-05-29 10:47:16 -0700</bug_when>
    <thetext>Created attachment 21415
Restore original behavior of isOSX()</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>81867</commentid>
    <comment_count>2</comment_count>
      <attachid>21415</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2008-05-29 10:57:47 -0700</bug_when>
    <thetext>Comment on attachment 21415
Restore original behavior of isOSX()

Seems fine.

Although I would probably use a different phrase from &quot;Apple port&quot;.

Please don&apos;t check in the double-ChangeLog entry.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>81869</commentid>
    <comment_count>3</comment_count>
    <who name="Kevin Ollivier">kevino</who>
    <bug_when>2008-05-29 11:04:11 -0700</bug_when>
    <thetext>landed in r34206, thanks! :)</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>21415</attachid>
            <date>2008-05-29 10:47:16 -0700</date>
            <delta_ts>2008-05-29 10:57:47 -0700</delta_ts>
            <desc>Restore original behavior of isOSX()</desc>
            <filename>platform-defines.patch</filename>
            <type>text/plain</type>
            <size>1564</size>
            <attacher name="Kevin Ollivier">kevino</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXRUb29scy9D
aGFuZ2VMb2cJKHJldmlzaW9uIDM0MjA1KQorKysgV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCSh3b3Jr
aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTEgQEAKKzIwMDgtMDUtMjkgIEtldmluIE9sbGl2aWVyICA8
a2V2aW5vQHRoZW9sbGl2aWVycy5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChP
T1BTISkuCisgICAgICAgIAorICAgICAgICBSZXN0b3JlIG9yaWdpbmFsIGJlaGF2aW9yIG9mIGlz
T1NYKCkgcmVmZXJyaW5nIHRvIHRoZSBBcHBsZSBwb3J0LCBub3QgdGhlIE9TIGl0c2VsZi4KKwor
ICAgICAgICAqIFNjcmlwdHMvd2Via2l0ZGlycy5wbToKKwogMjAwOC0wNS0yOSAgQWxleGV5IFBy
b3NrdXJ5YWtvdiAgPGFwQHdlYmtpdC5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgQWRhbSBS
b2Jlbi4KQEAgLTEyLDYgKzIwLDE0IEBACiAKIDIwMDgtMDUtMjcgIEtldmluIE9sbGl2aWVyICA8
a2V2aW5vQHRoZW9sbGl2aWVycy5jb20+CiAKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChP
T1BTISkuCisgICAgICAgIAorICAgICAgICBidWlsZCBzY3JpcHRzIG5lZWQgYSB3YXkgdG8gZGlm
ZmVyZW50aWF0ZSBiZXR3ZWVuIEFwcGxlIHBvcnQgYW5kIG90aGVycyBvbiBPUyBYLgorCisgICAg
ICAgICogU2NyaXB0cy93ZWJraXRkaXJzLnBtOgorCisyMDA4LTA1LTI3ICBLZXZpbiBPbGxpdmll
ciAgPGtldmlub0B0aGVvbGxpdmllcnMuY29tPgorCiAgICAgICAgIHd4IGJ1aWxkIGZpeC4gVXBk
YXRlIHRoZSBzYW1wbGUgYXBwIGFmdGVyIHd4V2ViRnJhbWUtPnd4V2ViQnJvd3NlclNoZWxsIHJl
bmFtZS4KIAogICAgICAgICAqIHd4L2Jyb3dzZXIvYnJvd3Nlci5jcHA6CkluZGV4OiBXZWJLaXRU
b29scy9TY3JpcHRzL3dlYmtpdGRpcnMucG0KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViS2l0VG9vbHMvU2Ny
aXB0cy93ZWJraXRkaXJzLnBtCShyZXZpc2lvbiAzNDIwMykKKysrIFdlYktpdFRvb2xzL1Njcmlw
dHMvd2Via2l0ZGlycy5wbQkod29ya2luZyBjb3B5KQpAQCAtNTEyLDExICs1MTIsMTkgQEAKICAg
ICByZXR1cm4gKCReTyBlcSAiY3lnd2luIikgfHwgMDsKIH0KIAotc3ViIGlzT1NYKCkKK3N1YiBp
c0RhcndpbigpCiB7CiAgICAgcmV0dXJuICgkXk8gZXEgImRhcndpbiIpIHx8IDA7CiB9CiAKKyMg
aXNPU1goKSBvbmx5IHJldHVybnMgdHJ1ZSBmb3IgQXBwbGUncyBwb3J0LCBub3QgZm9yIG90aGVy
IHBvcnRzIHRoYXQgY2FuIGJlCisjIGJ1aWx0L3J1biBvbiBPUyBYLgorc3ViIGlzT1NYKCkKK3sK
KyAgICByZXR1cm4gaXNEYXJ3aW4oKSB1bmxlc3MgKGlzUXQoKSBvciBpc0d0aygpIG9yIGlzV3go
KSk7CisgICAgcmV0dXJuIDA7Cit9CisKIHN1YiBkZXRlcm1pbmVPU1hWZXJzaW9uKCkKIHsKICAg
ICByZXR1cm4gaWYgJG9zWFZlcnNpb247Cg==
</data>
<flag name="review"
          id="9369"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>