<?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>156544</bug_id>
          
          <creation_ts>2016-04-13 09:58:11 -0700</creation_ts>
          <short_desc>FindWebP should not be misguided by pkg-config when cross-compiling.</short_desc>
          <delta_ts>2016-04-14 03:26:05 -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>Tools / Tests</component>
          <version>WebKit Local Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Konstantin Tokarev">annulen</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>cgarcia</cc>
    
    <cc>clopez</cc>
    
    <cc>commit-queue</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>lforschler</cc>
    
    <cc>mario</cc>
    
    <cc>mcatanzaro</cc>
    
    <cc>mrobinson</cc>
    
    <cc>ossy</cc>
    
    <cc>tpopela</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1183435</commentid>
    <comment_count>0</comment_count>
    <who name="Konstantin Tokarev">annulen</who>
    <bug_when>2016-04-13 09:58:11 -0700</bug_when>
    <thetext>We should use pkg-config output only as a hint, like other modules do.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1183436</commentid>
    <comment_count>1</comment_count>
      <attachid>276330</attachid>
    <who name="Konstantin Tokarev">annulen</who>
    <bug_when>2016-04-13 10:00:42 -0700</bug_when>
    <thetext>Created attachment 276330
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1183441</commentid>
    <comment_count>2</comment_count>
      <attachid>276331</attachid>
    <who name="Konstantin Tokarev">annulen</who>
    <bug_when>2016-04-13 10:31:41 -0700</bug_when>
    <thetext>Created attachment 276331
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1183450</commentid>
    <comment_count>3</comment_count>
      <attachid>276331</attachid>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2016-04-13 10:49:22 -0700</bug_when>
    <thetext>Comment on attachment 276331
Patch

By the way, do you have committer access yet? You can stop setting cq? and move it straight to cq+ once you do.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1183456</commentid>
    <comment_count>4</comment_count>
    <who name="Konstantin Tokarev">annulen</who>
    <bug_when>2016-04-13 10:57:22 -0700</bug_when>
    <thetext>Not yet.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1183572</commentid>
    <comment_count>5</comment_count>
      <attachid>276331</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2016-04-13 15:58:22 -0700</bug_when>
    <thetext>Comment on attachment 276331
Patch

Clearing flags on attachment: 276331

Committed r199518: &lt;http://trac.webkit.org/changeset/199518&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1183573</commentid>
    <comment_count>6</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2016-04-13 15:58:28 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1183603</commentid>
    <comment_count>7</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2016-04-13 18:06:41 -0700</bug_when>
    <thetext>Continuous build broke with this.

I have triggered clean builds on all the GTK+ bots.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1183674</commentid>
    <comment_count>8</comment_count>
    <who name="Konstantin Tokarev">annulen</who>
    <bug_when>2016-04-14 02:47:10 -0700</bug_when>
    <thetext>(In reply to comment #7)
&gt; Continuous build broke with this.
&gt; 
&gt; I have triggered clean builds on all the GTK+ bots.

This patch requires one clean build after landing, after that continuous builds should work fine (at least it works in my local tests).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1183678</commentid>
    <comment_count>9</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2016-04-14 03:26:05 -0700</bug_when>
    <thetext>(In reply to comment #8)
&gt; (In reply to comment #7)
&gt; &gt; Continuous build broke with this.
&gt; &gt; 
&gt; &gt; I have triggered clean builds on all the GTK+ bots.
&gt; 
&gt; This patch requires one clean build after landing, after that continuous
&gt; builds should work fine (at least it works in my local tests).

Yes.

that is what I wanted to say :)</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>276330</attachid>
            <date>2016-04-13 10:00:42 -0700</date>
            <delta_ts>2016-04-13 10:31:30 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-156544-20160413200052.patch</filename>
            <type>text/plain</type>
            <size>2095</size>
            <attacher name="Konstantin Tokarev">annulen</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTk4NjY2CmRpZmYgLS1naXQgYS9Tb3VyY2UvY21ha2UvRmlu
ZFdlYlAuY21ha2UgYi9Tb3VyY2UvY21ha2UvRmluZFdlYlAuY21ha2UKaW5kZXggYTYyN2IxMzhj
ZmRmOTI1OGUyYzdiNjI2ZTY1ZmE1Y2U0ODE4NjFiNS4uNjc3MjM1MmQ5NTcxZDk5MzgzN2YyZWZj
MTY5NmJmNDI2NGRjNGQwMCAxMDA2NDQKLS0tIGEvU291cmNlL2NtYWtlL0ZpbmRXZWJQLmNtYWtl
CisrKyBiL1NvdXJjZS9jbWFrZS9GaW5kV2ViUC5jbWFrZQpAQCAtMzAsMTkgKzMwLDIyIEBACiAj
IEFEVklTRUQgT0YgVEhFIFBPU1NJQklMSVRZIE9GIFNVQ0ggREFNQUdFLgogCiBmaW5kX3BhY2th
Z2UoUGtnQ29uZmlnKQotcGtnX2NoZWNrX21vZHVsZXMoV0VCUCBsaWJ3ZWJwKQorcGtnX2NoZWNr
X21vZHVsZXMoUENfV0VCUCBRVUlFVCBsaWJ3ZWJwKQogCi1pZiAoTk9UKFdFQlBfRk9VTkQpKQot
ICAgICMgT2xkZXIgaW5zdGFsbGF0aW9ucyBvZiBsaWJ3ZWJwIGRvIG5vdCBpbnN0YWxsIGEgcGtn
Y29uZmlnIGZpbGUsIHNvIHdlIGZhbGwKLSAgICAjIGJhY2sgdG8gYSBtYW51YWwgc2VhcmNoIGZv
ciBhIGxpYndlYnAgaGVhZGVyIGZpbGUuCi0gICAgaW5jbHVkZShDaGVja0luY2x1ZGVGaWxlcykK
LSAgICBjaGVja19pbmNsdWRlX2ZpbGVzKCJ3ZWJwL2RlY29kZS5oIiBXRUJQX0ZPVU5EX0hFQURF
UikKLSAgICBpZiAoV0VCUF9GT1VORF9IRUFERVIpCi0gICAgICAgIHNldChXRUJQX0xJQlJBUklF
UyAiLWx3ZWJwIikKLSAgICAgICAgc2V0KFdFQlBfRk9VTkQgVFJVRSkKLSAgICBlbmRpZiAoKQot
ZW5kaWYgKCkKKyMgTG9vayBmb3IgdGhlIGhlYWRlciBmaWxlLgorZmluZF9wYXRoKFdFQlBfSU5D
TFVERV9ESVJTCisgICAgTkFNRVMgd2VicC9kZWNvZGUuaAorICAgIEhJTlRTICR7UENfV0VCUF9J
TkNMVURFRElSfSAke1BDX1dFQlBfSU5DTFVERV9ESVJTfQorKQorbWFya19hc19hZHZhbmNlZChX
RUJQX0lOQ0xVREVfRElSUykKKworIyBMb29rIGZvciB0aGUgbGlicmFyeS4KK2ZpbmRfbGlicmFy
eSgKKyAgICBXRUJQX0xJQlJBUklFUworICAgIE5BTUVTIHdlYnAKKyAgICBISU5UUyAke1BDX1dF
QlBfTElCRElSfSAke1BDX1dFQlBfTElCUkFSWV9ESVJTfQorKQorbWFya19hc19hZHZhbmNlZChX
RUJQX0xJQlJBUklFUykKIAotIyBXRUJQX0lOQ0xVREVfRElSUyBpcyBvZnRlbiBlbXB0eSwgc28g
d2UgcmVseSBvbmx5IG9uIFdFQlBfTElCUkFSSUVTLgogaW5jbHVkZShGaW5kUGFja2FnZUhhbmRs
ZVN0YW5kYXJkQXJncykKLUZJTkRfUEFDS0FHRV9IQU5ETEVfU1RBTkRBUkRfQVJHUyhXRUJQIERF
RkFVTFRfTVNHIFdFQlBfTElCUkFSSUVTKQorRklORF9QQUNLQUdFX0hBTkRMRV9TVEFOREFSRF9B
UkdTKFdFQlAgREVGQVVMVF9NU0cgV0VCUF9JTkNMVURFX0RJUlMgV0VCUF9MSUJSQVJJRVMpCmRp
ZmYgLS1naXQgYS9DaGFuZ2VMb2cgYi9DaGFuZ2VMb2cKaW5kZXggMjYyNGIwZDJmOWViMWIwNWFk
ZDU0NDFmYTg0YTFkNDE3ZWE2NjA3YS4uMjQ0ZTA4Nzc3YTQ3NDVjOWQyZTZmMzIwZjVlZDI1YWNk
NjYxZDJiMyAxMDA2NDQKLS0tIGEvQ2hhbmdlTG9nCisrKyBiL0NoYW5nZUxvZwpAQCAtMSwzICsx
LDE0IEBACisyMDE2LTA0LTEzICBLb25zdGFudGluIFRva2FyZXYgIDxhbm51bGVuQHlhbmRleC5y
dT4KKworICAgICAgICBGaW5kV2ViUCBzaG91bGQgbm90IGJlIG1pc2d1aWRlZCBieSBwa2ctY29u
ZmlnIHdoZW4gY3Jvc3MtY29tcGlsaW5nLgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9y
Zy9zaG93X2J1Zy5jZ2k/aWQ9MTU2NTQ0CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChP
T1BTISkuCisKKyAgICAgICAgV2Ugc2hvdWxkIHVzZSBwa2ctY29uZmlnIG91dHB1dCBvbmx5IGFz
IGEgaGludCwgbGlrZSBvdGhlciBtb2R1bGVzIGRvLgorCisgICAgICAgICogU291cmNlL2NtYWtl
L0ZpbmRXZWJQLmNtYWtlOgorCiAyMDE2LTAzLTE4ICBZdXN1a2UgU3V6dWtpICA8dXRhdGFuZS50
ZWFAZ21haWwuY29tPgogCiAgICAgICAgIFVucmV2aWV3ZWQsIG1pc3NpbmcgU0VUX0FORF9FWFBP
U0VfVE9fQlVJTEQgZm9yIFVTRV9VRElTODYgaW4gSlNDT25seSBwb3J0Cg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>276331</attachid>
            <date>2016-04-13 10:31:41 -0700</date>
            <delta_ts>2016-04-13 15:58:22 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-156544-20160413203151.patch</filename>
            <type>text/plain</type>
            <size>2095</size>
            <attacher name="Konstantin Tokarev">annulen</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTk4NjY2CmRpZmYgLS1naXQgYS9Tb3VyY2UvY21ha2UvRmlu
ZFdlYlAuY21ha2UgYi9Tb3VyY2UvY21ha2UvRmluZFdlYlAuY21ha2UKaW5kZXggYTYyN2IxMzhj
ZmRmOTI1OGUyYzdiNjI2ZTY1ZmE1Y2U0ODE4NjFiNS4uNjc3MjM1MmQ5NTcxZDk5MzgzN2YyZWZj
MTY5NmJmNDI2NGRjNGQwMCAxMDA2NDQKLS0tIGEvU291cmNlL2NtYWtlL0ZpbmRXZWJQLmNtYWtl
CisrKyBiL1NvdXJjZS9jbWFrZS9GaW5kV2ViUC5jbWFrZQpAQCAtMzAsMTkgKzMwLDIyIEBACiAj
IEFEVklTRUQgT0YgVEhFIFBPU1NJQklMSVRZIE9GIFNVQ0ggREFNQUdFLgogCiBmaW5kX3BhY2th
Z2UoUGtnQ29uZmlnKQotcGtnX2NoZWNrX21vZHVsZXMoV0VCUCBsaWJ3ZWJwKQorcGtnX2NoZWNr
X21vZHVsZXMoUENfV0VCUCBRVUlFVCBsaWJ3ZWJwKQogCi1pZiAoTk9UKFdFQlBfRk9VTkQpKQot
ICAgICMgT2xkZXIgaW5zdGFsbGF0aW9ucyBvZiBsaWJ3ZWJwIGRvIG5vdCBpbnN0YWxsIGEgcGtn
Y29uZmlnIGZpbGUsIHNvIHdlIGZhbGwKLSAgICAjIGJhY2sgdG8gYSBtYW51YWwgc2VhcmNoIGZv
ciBhIGxpYndlYnAgaGVhZGVyIGZpbGUuCi0gICAgaW5jbHVkZShDaGVja0luY2x1ZGVGaWxlcykK
LSAgICBjaGVja19pbmNsdWRlX2ZpbGVzKCJ3ZWJwL2RlY29kZS5oIiBXRUJQX0ZPVU5EX0hFQURF
UikKLSAgICBpZiAoV0VCUF9GT1VORF9IRUFERVIpCi0gICAgICAgIHNldChXRUJQX0xJQlJBUklF
UyAiLWx3ZWJwIikKLSAgICAgICAgc2V0KFdFQlBfRk9VTkQgVFJVRSkKLSAgICBlbmRpZiAoKQot
ZW5kaWYgKCkKKyMgTG9vayBmb3IgdGhlIGhlYWRlciBmaWxlLgorZmluZF9wYXRoKFdFQlBfSU5D
TFVERV9ESVJTCisgICAgTkFNRVMgd2VicC9kZWNvZGUuaAorICAgIEhJTlRTICR7UENfV0VCUF9J
TkNMVURFRElSfSAke1BDX1dFQlBfSU5DTFVERV9ESVJTfQorKQorbWFya19hc19hZHZhbmNlZChX
RUJQX0lOQ0xVREVfRElSUykKKworIyBMb29rIGZvciB0aGUgbGlicmFyeS4KK2ZpbmRfbGlicmFy
eSgKKyAgICBXRUJQX0xJQlJBUklFUworICAgIE5BTUVTIHdlYnAKKyAgICBISU5UUyAke1BDX1dF
QlBfTElCRElSfSAke1BDX1dFQlBfTElCUkFSWV9ESVJTfQorKQorbWFya19hc19hZHZhbmNlZChX
RUJQX0xJQlJBUklFUykKIAotIyBXRUJQX0lOQ0xVREVfRElSUyBpcyBvZnRlbiBlbXB0eSwgc28g
d2UgcmVseSBvbmx5IG9uIFdFQlBfTElCUkFSSUVTLgogaW5jbHVkZShGaW5kUGFja2FnZUhhbmRs
ZVN0YW5kYXJkQXJncykKLUZJTkRfUEFDS0FHRV9IQU5ETEVfU1RBTkRBUkRfQVJHUyhXRUJQIERF
RkFVTFRfTVNHIFdFQlBfTElCUkFSSUVTKQorRklORF9QQUNLQUdFX0hBTkRMRV9TVEFOREFSRF9B
UkdTKFdFQlAgREVGQVVMVF9NU0cgV0VCUF9JTkNMVURFX0RJUlMgV0VCUF9MSUJSQVJJRVMpCmRp
ZmYgLS1naXQgYS9DaGFuZ2VMb2cgYi9DaGFuZ2VMb2cKaW5kZXggMjYyNGIwZDJmOWViMWIwNWFk
ZDU0NDFmYTg0YTFkNDE3ZWE2NjA3YS4uMjQ0ZTA4Nzc3YTQ3NDVjOWQyZTZmMzIwZjVlZDI1YWNk
NjYxZDJiMyAxMDA2NDQKLS0tIGEvQ2hhbmdlTG9nCisrKyBiL0NoYW5nZUxvZwpAQCAtMSwzICsx
LDE0IEBACisyMDE2LTA0LTEzICBLb25zdGFudGluIFRva2FyZXYgIDxhbm51bGVuQHlhbmRleC5y
dT4KKworICAgICAgICBGaW5kV2ViUCBzaG91bGQgbm90IGJlIG1pc2d1aWRlZCBieSBwa2ctY29u
ZmlnIHdoZW4gY3Jvc3MtY29tcGlsaW5nLgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9y
Zy9zaG93X2J1Zy5jZ2k/aWQ9MTU2NTQ0CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChP
T1BTISkuCisKKyAgICAgICAgV2Ugc2hvdWxkIHVzZSBwa2ctY29uZmlnIG91dHB1dCBvbmx5IGFz
IGEgaGludCwgbGlrZSBvdGhlciBtb2R1bGVzIGRvLgorCisgICAgICAgICogU291cmNlL2NtYWtl
L0ZpbmRXZWJQLmNtYWtlOgorCiAyMDE2LTAzLTE4ICBZdXN1a2UgU3V6dWtpICA8dXRhdGFuZS50
ZWFAZ21haWwuY29tPgogCiAgICAgICAgIFVucmV2aWV3ZWQsIG1pc3NpbmcgU0VUX0FORF9FWFBP
U0VfVE9fQlVJTEQgZm9yIFVTRV9VRElTODYgaW4gSlNDT25seSBwb3J0Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>