<?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>177273</bug_id>
          
          <creation_ts>2017-09-20 14:43:50 -0700</creation_ts>
          <short_desc>REGRESSION(r222194) [WPE] Library and executable output names broken</short_desc>
          <delta_ts>2017-09-27 12:16:39 -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>WPE WebKit</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>
          
          <blocked>174558</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Carlos Alberto Lopez Perez">clopez</reporter>
          <assigned_to name="Michael Catanzaro">mcatanzaro</assigned_to>
          <cc>bugs-noreply</cc>
    
    <cc>cgarcia</cc>
    
    <cc>commit-queue</cc>
    
    <cc>mcatanzaro</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>zan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1350996</commentid>
    <comment_count>0</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2017-09-20 14:43:50 -0700</bug_when>
    <thetext>Not sure what caused this but it was something recent.

$ Tools/Scripts/run-minibrowser --wpe 
Can&apos;t find built framework at &quot;/home/clopez/webkit/wpe/WebKitBuild/Release/lib/libWPEWebKit.so&quot;.


And if I try this patch
--- a/Tools/Scripts/webkitdirs.pm
+++ b/Tools/Scripts/webkitdirs.pm
@@ -959,7 +959,7 @@ sub builtDylibPathForName
         }
     }
     if (isWPE()) {
-        return &quot;$configurationProductDir/lib/libWPEWebKit.so&quot;;
+        return &quot;$configurationProductDir/lib/libWebKit.so&quot;;
     }
 
     die &quot;Unsupported platform, can&apos;t determine built library locations.\nTry `build-webkit --help` for more information.\n&quot;;



Then i get:

$ Tools/Scripts/run-minibrowser --wpe 
Starting dyz 
lib/wpe.lua:3: libWPEWebKit.so: cannot open shared object file: No such file or directory</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1351001</commentid>
    <comment_count>1</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2017-09-20 14:51:48 -0700</bug_when>
    <thetext>This was caused by r222194 &lt;https://trac.webkit.org/r222194&gt;

I reverted it locally, triggered a clean build and now I get libWPEWebKit.so generated correctly and the minibrowser works back as expected.

BTW.. I have just triggered clean builds on the WPE bots, as I suspect they were running all this time using the old libWPEWebKit.so (from &lt;r222194)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1351018</commentid>
    <comment_count>2</comment_count>
      <attachid>321379</attachid>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2017-09-20 15:10:06 -0700</bug_when>
    <thetext>Created attachment 321379
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1351028</commentid>
    <comment_count>3</comment_count>
      <attachid>321379</attachid>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2017-09-20 15:35:47 -0700</bug_when>
    <thetext>Comment on attachment 321379
Patch

Tested. it works ok.
Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1351043</commentid>
    <comment_count>4</comment_count>
      <attachid>321379</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-09-20 16:05:27 -0700</bug_when>
    <thetext>Comment on attachment 321379
Patch

Clearing flags on attachment: 321379

Committed r222296: &lt;http://trac.webkit.org/changeset/222296&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1351044</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-09-20 16:05:30 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>321379</attachid>
            <date>2017-09-20 15:10:06 -0700</date>
            <delta_ts>2017-09-20 16:05:27 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-177273-20170920171005.patch</filename>
            <type>text/plain</type>
            <size>2733</size>
            <attacher name="Michael Catanzaro">mcatanzaro</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjIyMjkyCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IDFkNGM5ZmU1YmFmNGNjYTZm
MjdhOGFmNWFjMzNiY2Y2MTA0YzZlMWMuLjlmNWU2Nzg0YTg4MzcwZTlmMDFlYTNhZDQwZDhlNGQ4
ZjNkYmNmNGIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTctMDktMjAgIE1pY2hhZWwg
Q2F0YW56YXJvICA8bWNhdGFuemFyb0BpZ2FsaWEuY29tPgorCisgICAgICAgIFJFR1JFU1NJT04o
cjIyMjE5NCkgW1dQRV0gTGlicmFyeSBhbmQgZXhlY3V0YWJsZSBvdXRwdXQgbmFtZXMgYnJva2Vu
CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNzcyNzMK
KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBNb3ZlIHRo
ZSBXZWJLaXQyIG91dHB1dCBuYW1lIHZhcmlhYmxlcyB0byBoZXJlIGZyb20gU291cmNlL2NtYWtl
LCBzaW5jZSB0aGlzIGlzIGEgYmV0dGVyCisgICAgICAgIHBsYWNlIGFuZCB3aGVyZSB0aGV5IGFy
ZSBzZXQgb24gYWxsIG90aGVyIHBvcnRzLiBVcGRhdGUgdGhlbSBmb3IgdGhlIGNoYW5nZXMgaW4g
cjIyMjE5NC4KKworICAgICAgICAqIFBsYXRmb3JtV1BFLmNtYWtlOgorCiAyMDE3LTA5LTIwICBK
b25hdGhhbiBCZWRhcmQgIDxqYmVkYXJkQGFwcGxlLmNvbT4KIAogICAgICAgICBSZW1vdmVkIG51
bGxhYmxlIGZyb20gVUlEcmFnSXRlbSBwcm9wZXJ0eQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktp
dC9QbGF0Zm9ybVdQRS5jbWFrZSBiL1NvdXJjZS9XZWJLaXQvUGxhdGZvcm1XUEUuY21ha2UKaW5k
ZXggNjJmODlkYzc3MDBkMDA4ZjY2NWMzZjFhNTU5NzY3MjM2NGE4YmU1ZS4uNjdlYzU0Yjk2N2Rm
MzNjOWQxMDJiMTc4NzZiYjY4OTI3ZmU1NmU1MiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdC9Q
bGF0Zm9ybVdQRS5jbWFrZQorKysgYi9Tb3VyY2UvV2ViS2l0L1BsYXRmb3JtV1BFLmNtYWtlCkBA
IC0xLDMgKzEsOCBAQAorc2V0KFdlYktpdF9PVVRQVVRfTkFNRSBXUEVXZWJLaXQpCitzZXQoV2Vi
S2l0X1dlYlByb2Nlc3NfT1VUUFVUX05BTUUgV1BFV2ViUHJvY2VzcykKK3NldChXZWJLaXRfTmV0
d29ya1Byb2Nlc3NfT1VUUFVUX05BTUUgV1BFTmV0d29ya1Byb2Nlc3MpCitzZXQoV2ViS2l0X1N0
b3JhZ2VQcm9jZXNzX09VVFBVVF9OQU1FIFdQRVN0b3JhZ2VQcm9jZXNzKQorCiBmaWxlKE1BS0Vf
RElSRUNUT1JZICR7REVSSVZFRF9TT1VSQ0VTX1dQRV9BUElfRElSfSkKIGZpbGUoTUFLRV9ESVJF
Q1RPUlkgJHtGT1JXQVJESU5HX0hFQURFUlNfV1BFX0RJUn0pCiBmaWxlKE1BS0VfRElSRUNUT1JZ
ICR7Rk9SV0FSRElOR19IRUFERVJTX1dQRV9FWFRFTlNJT05fRElSfSkKZGlmZiAtLWdpdCBhL1Nv
dXJjZS9jbWFrZS9PcHRpb25zV1BFLmNtYWtlIGIvU291cmNlL2NtYWtlL09wdGlvbnNXUEUuY21h
a2UKaW5kZXggMDI1YTg0MWJhMDliMTU5ZDdlYmMzNjI1NjgxMDViNGQyZmQxMzFjNC4uMWJiYjg5
OWU3ZTdkNzY0YmI1ZTUxNmQ4ZmYwMDg4ZGZiNjg2OTU5ZSAxMDA2NDQKLS0tIGEvU291cmNlL2Nt
YWtlL09wdGlvbnNXUEUuY21ha2UKKysrIGIvU291cmNlL2NtYWtlL09wdGlvbnNXUEUuY21ha2UK
QEAgLTU0LDEwICs1NCw2IEBAIHNldChFTkFCTEVfQVBJX1RFU1RTICR7REVWRUxPUEVSX01PREV9
KQogCiBzZXQoSmF2YVNjcmlwdENvcmVfTElCUkFSWV9UWVBFIFNUQVRJQykKIHNldChXZWJDb3Jl
X0xJQlJBUllfVFlQRSBTVEFUSUMpCi1zZXQoV2ViS2l0Ml9PVVRQVVRfTkFNRSBXUEVXZWJLaXQp
Ci1zZXQoV2ViS2l0Ml9XZWJQcm9jZXNzX09VVFBVVF9OQU1FIFdQRVdlYlByb2Nlc3MpCi1zZXQo
V2ViS2l0Ml9OZXR3b3JrUHJvY2Vzc19PVVRQVVRfTkFNRSBXUEVOZXR3b3JrUHJvY2VzcykKLXNl
dChXZWJLaXQyX1N0b3JhZ2VQcm9jZXNzX09VVFBVVF9OQU1FIFdQRVN0b3JhZ2VQcm9jZXNzKQog
CiBmaW5kX3BhY2thZ2UoSUNVIFJFUVVJUkVEKQogZmluZF9wYWNrYWdlKFRocmVhZHMgUkVRVUlS
RUQpCmRpZmYgLS1naXQgYS9DaGFuZ2VMb2cgYi9DaGFuZ2VMb2cKaW5kZXggNWFjZmEwZTFiYmM2
YTRlODBkN2ZlYWZmNjUwNjdlNDQ0MTEyMTRhMi4uYzVkZDhlNTI5MTU2ODgwOGUyYjRlM2ZlMDIx
NzAyZDk1MDhkZDJhOSAxMDA2NDQKLS0tIGEvQ2hhbmdlTG9nCisrKyBiL0NoYW5nZUxvZwpAQCAt
MSwzICsxLDE0IEBACisyMDE3LTA5LTIwICBNaWNoYWVsIENhdGFuemFybyAgPG1jYXRhbnphcm9A
aWdhbGlhLmNvbT4KKworICAgICAgICBSRUdSRVNTSU9OKHIyMjIxOTQpIFtXUEVdIExpYnJhcnkg
YW5kIGV4ZWN1dGFibGUgb3V0cHV0IG5hbWVzIGJyb2tlbgorICAgICAgICBodHRwczovL2J1Z3Mu
d2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTc3MjczCisKKyAgICAgICAgUmV2aWV3ZWQgYnkg
Tk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgTW92ZSB0aGUgV2ViS2l0MiBvdXRwdXQgbmFtZSB2
YXJpYWJsZXMgZnJvbSBoZXJlIHRvIFNvdXJjZS9XZWJLaXQuCisKKyAgICAgICAgKiBTb3VyY2Uv
Y21ha2UvT3B0aW9uc1dQRS5jbWFrZToKKwogMjAxNy0wOS0yMCAgQWxiZXJ0byBHYXJjaWEgIDxi
ZXJ0b0BpZ2FsaWEuY29tPgogCiAgICAgICAgIEZpeCBIUFBBIGFuZCBBbHBoYSBidWlsZHMK
</data>

          </attachment>
      

    </bug>

</bugzilla>