<?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>39411</bug_id>
          
          <creation_ts>2010-05-20 00:59:43 -0700</creation_ts>
          <short_desc>[BREWMP] Don&apos;t include POSIX headers in WebCorePrefix.h</short_desc>
          <delta_ts>2010-06-12 20:41:34 -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>WebCore Misc.</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Other</rep_platform>
          <op_sys>Other</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>33564</blocked>
          <everconfirmed>0</everconfirmed>
          <reporter name="Kwang Yul Seo">skyul</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>beergun</cc>
    
    <cc>commit-queue</cc>
    
    <cc>joybro201</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>228243</commentid>
    <comment_count>0</comment_count>
    <who name="Kwang Yul Seo">skyul</who>
    <bug_when>2010-05-20 00:59:43 -0700</bug_when>
    <thetext>When building for BREW MP, don&apos;t include POSIX headers: &lt;fcntl.h&gt;, &lt;pthread.h&gt; and &lt;sys/types.h&gt;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>228246</commentid>
    <comment_count>1</comment_count>
      <attachid>56571</attachid>
    <who name="Kwang Yul Seo">skyul</who>
    <bug_when>2010-05-20 01:03:18 -0700</bug_when>
    <thetext>Created attachment 56571
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>229983</commentid>
    <comment_count>2</comment_count>
      <attachid>56571</attachid>
    <who name="Laszlo Gombos">laszlo.gombos</who>
    <bug_when>2010-05-24 16:56:22 -0700</bug_when>
    <thetext>Comment on attachment 56571
Patch

&gt; +#if !defined(BUILDING_BREWMP__)
&gt;  #include &lt;pthread.h&gt;
&gt; +#endif

Can we use HAVE(PTHREAD_H) as a guard here instead ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>230163</commentid>
    <comment_count>3</comment_count>
    <who name="Kwang Yul Seo">skyul</who>
    <bug_when>2010-05-25 01:44:57 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; Can we use HAVE(PTHREAD_H) as a guard here instead ?

I will try HAVE(XXX_H) and see if other ports build OK.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>230170</commentid>
    <comment_count>4</comment_count>
    <who name="Kwang Yul Seo">skyul</who>
    <bug_when>2010-05-25 01:54:45 -0700</bug_when>
    <thetext>(In reply to comment #3)
&gt; I will try HAVE(XXX_H) and see if other ports build OK.

Oops. It seems we can&apos;t use HAVE(WTF_FEATURE) macro in WebCorePrefix.h.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>237415</commentid>
    <comment_count>5</comment_count>
      <attachid>56571</attachid>
    <who name="Kent Tamura">tkent</who>
    <bug_when>2010-06-12 19:04:44 -0700</bug_when>
    <thetext>Comment on attachment 56571
Patch

Looks OK.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>237452</commentid>
    <comment_count>6</comment_count>
      <attachid>56571</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-06-12 20:41:29 -0700</bug_when>
    <thetext>Comment on attachment 56571
Patch

Clearing flags on attachment: 56571

Committed r61074: &lt;http://trac.webkit.org/changeset/61074&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>237453</commentid>
    <comment_count>7</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-06-12 20:41:34 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>56571</attachid>
            <date>2010-05-20 01:03:18 -0700</date>
            <delta_ts>2010-06-12 20:41:28 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>posix.patch</filename>
            <type>text/plain</type>
            <size>1490</size>
            <attacher name="Kwang Yul Seo">skyul</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA1OTgyMykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTUgQEAKKzIwMTAtMDUtMjAgIEt3YW5nIFl1bCBTZW8gIDxza3l1bEBjb21wYW55
MTAwLm5ldD4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAg
ICBbQlJFV01QXSBEb24ndCBpbmNsdWRlIFBPU0lYIGhlYWRlcnMgaW4gV2ViQ29yZVByZWZpeC5o
CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zOTQxMQor
CisgICAgICAgIFdoZW4gYnVpbGRpbmcgZm9yIEJSRVcgTVAsIGRvbid0IGluY2x1ZGUgUE9TSVgg
aGVhZGVyczoKKyAgICAgICAgPGZjbnRsLmg+LCA8cHRocmVhZC5oPiBhbmQgPHN5cy90eXBlcy5o
Pi4KKworICAgICAgICAqIFdlYkNvcmVQcmVmaXguaDoKKwogMjAxMC0wNS0xOSAgVmFuZ2VsaXMg
S29ra2V2aXMgIDx2YW5nZWxpc0BjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkg
RGFyaW4gRmlzaGVyLgpJbmRleDogV2ViQ29yZS9XZWJDb3JlUHJlZml4LmgKPT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQot
LS0gV2ViQ29yZS9XZWJDb3JlUHJlZml4LmgJKHJldmlzaW9uIDU5ODIzKQorKysgV2ViQ29yZS9X
ZWJDb3JlUHJlZml4LmgJKHdvcmtpbmcgY29weSkKQEAgLTU5LDcgKzU5LDkgQEAKICNlbmRpZgog
CiAjZWxzZQorI2lmICFkZWZpbmVkKEJVSUxESU5HX0JSRVdNUF9fKQogI2luY2x1ZGUgPHB0aHJl
YWQuaD4KKyNlbmRpZgogI2VuZGlmIC8vIGRlZmluZWQoV0lOMzIpIHx8IGRlZmluZWQoX1dJTjMy
KQogCiAjaWYgZGVmaW5lZChBTkRST0lEKQpAQCAtNzUsOCArNzcsMTAgQEAKICNpbmNsdWRlIDxK
YXZhU2NyaXB0Q29yZS9jb25maWcuaD4KICNlbmRpZgogCisjaWYgIWRlZmluZWQoQlVJTERJTkdf
QlJFV01QX18pCiAjaW5jbHVkZSA8c3lzL3R5cGVzLmg+CiAjaW5jbHVkZSA8ZmNudGwuaD4KKyNl
bmRpZgogI2lmIGRlZmluZWQoX19BUFBMRV9fKQogI2luY2x1ZGUgPHJlZ2V4Lmg+CiAjZW5kaWYK
QEAgLTEwNiwxMSArMTEwLDE1IEBACiAKICNlbmRpZgogCisjaWYgIWRlZmluZWQoQlVJTERJTkdf
QlJFV01QX18pCiAjaW5jbHVkZSA8c3lzL3R5cGVzLmg+CisjZW5kaWYKICNpZiBkZWZpbmVkKF9f
QVBQTEVfXykKICNpbmNsdWRlIDxzeXMvcGFyYW0uaD4KICNlbmRpZgorI2lmICFkZWZpbmVkKEJV
SUxESU5HX0JSRVdNUF9fKQogI2luY2x1ZGUgPHN5cy9zdGF0Lmg+CisjZW5kaWYKICNpZiBkZWZp
bmVkKF9fQVBQTEVfXykKICNpbmNsdWRlIDxzeXMvdGltZS5oPgogI2luY2x1ZGUgPHN5cy9yZXNv
dXJjZS5oPgo=
</data>

          </attachment>
      

    </bug>

</bugzilla>