<?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>170426</bug_id>
          
          <creation_ts>2017-04-03 15:28:09 -0700</creation_ts>
          <short_desc>[GTK][JHBuild] Fetch libvpx from a release tarball instead of git</short_desc>
          <delta_ts>2017-04-03 16:40:54 -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>WebKitGTK</component>
          <version>WebKit Nightly 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="Carlos Alberto Lopez Perez">clopez</reporter>
          <assigned_to name="Carlos Alberto Lopez Perez">clopez</assigned_to>
          <cc>bugs-noreply</cc>
    
    <cc>calvaris</cc>
    
    <cc>cgarcia</cc>
    
    <cc>mcatanzaro</cc>
    
    <cc>zan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1293862</commentid>
    <comment_count>0</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2017-04-03 15:28:09 -0700</bug_when>
    <thetext>I don&apos;t know what is the issue with the libvpx repository, but when using the JHBUILD_MIRROR feature &lt;https://trac.webkit.org/changeset/166061&gt; it sometimes takes ages to run the fetch operation or it even fails with a network timeout.

And, I see no reason to use the git repository.

Using a release tarball should be better in any case and will lower the disk and network needs required for building WebKitGTK.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1293883</commentid>
    <comment_count>1</comment_count>
      <attachid>306134</attachid>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2017-04-03 15:53:57 -0700</bug_when>
    <thetext>Created attachment 306134
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1293902</commentid>
    <comment_count>2</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2017-04-03 16:35:08 -0700</bug_when>
    <thetext>Thanks for the review.

I&apos;m landing it with a minor diff to use the variable version on the filename.


-    &lt;branch repo=&quot;downloads.webmproject.org&quot;
-            module=&quot;libvpx-1.6.0.tar.bz2&quot; version=&quot;1.6.0&quot;
+    &lt;branch version=&quot;1.6.0&quot;
+            module=&quot;libvpx-${version}.tar.bz2&quot;
+            repo=&quot;downloads.webmproject.org&quot;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1293903</commentid>
    <comment_count>3</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2017-04-03 16:40:19 -0700</bug_when>
    <thetext>Committed r214854: &lt;http://trac.webkit.org/changeset/214854&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>306134</attachid>
            <date>2017-04-03 15:53:57 -0700</date>
            <delta_ts>2017-04-03 16:40:54 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-170426-20170404005355.patch</filename>
            <type>text/plain</type>
            <size>2013</size>
            <attacher name="Carlos Alberto Lopez Perez">clopez</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjE0ODQ3CmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg
Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggZDU2ZTc0ZDI1OGZlZjIxZWVmZTk1ODA2MmZhODk2ZTEz
M2NhMzU0Ni4uZWJlZTZmZmViZGU5NGJiNDUyNmVjMGQxNWU0NzI1ZTM0MWVhYjgwNSAxMDA2NDQK
LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEy
IEBACisyMDE3LTA0LTAzICBDYXJsb3MgQWxiZXJ0byBMb3BleiBQZXJleiAgPGNsb3BlekBpZ2Fs
aWEuY29tPgorCisgICAgICAgIFtHVEtdW0pIQnVpbGRdIEZldGNoIGxpYnZweCBmcm9tIGEgcmVs
ZWFzZSB0YXJiYWxsIGluc3RlYWQgb2YgZ2l0CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQu
b3JnL3Nob3dfYnVnLmNnaT9pZD0xNzA0MjYKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkg
KE9PUFMhKS4KKworICAgICAgICAqIGd0ay9qaGJ1aWxkLm1vZHVsZXM6CisKIDIwMTctMDQtMDMg
IEJyaWFuIEJ1cmcgIDxiYnVyZ0BhcHBsZS5jb20+CiAKICAgICAgICAgcnVuLXNhZmFyaSBzaG91
bGQgc3VwcG9ydCBsYXVuY2hpbmcgd2l0aCBjdXN0b20gbGFuZ3VhZ2UgYW5kIGxvY2FsZQpkaWZm
IC0tZ2l0IGEvVG9vbHMvZ3RrL2poYnVpbGQubW9kdWxlcyBiL1Rvb2xzL2d0ay9qaGJ1aWxkLm1v
ZHVsZXMKaW5kZXggOTlhYWVhZDY0OTNiZTY4NTcxMzIzZjYzYTA5YmI4ODc4ZmFiN2QyNC4uOGY3
ZjA2NTg2YzMxYzU1NTFiOTg2ZTNiOWJlZTYzOWU3YmJjNzgzYSAxMDA2NDQKLS0tIGEvVG9vbHMv
Z3RrL2poYnVpbGQubW9kdWxlcworKysgYi9Ub29scy9ndGsvamhidWlsZC5tb2R1bGVzCkBAIC04
MCw2ICs4MCw4IEBACiAgICAgICBocmVmPSJodHRwOi8vbGx2bS5vcmciLz4KICAgPHJlcG9zaXRv
cnkgdHlwZT0idGFyYmFsbCIgbmFtZT0id2Via2l0Z3RrLWpoYnVpbGQtbWlycm9yIgogICAgICAg
aHJlZj0iaHR0cDovL3dlYmtpdGd0ay5vcmcvamhidWlsZF9taXJyb3IvIi8+CisgIDxyZXBvc2l0
b3J5IHR5cGU9InRhcmJhbGwiIG5hbWU9ImRvd25sb2Fkcy53ZWJtcHJvamVjdC5vcmciCisgICAg
ICBocmVmPSJodHRwOi8vZG93bmxvYWRzLndlYm1wcm9qZWN0Lm9yZy9yZWxlYXNlcy93ZWJtLyIv
PgogCiAgIDxhdXRvdG9vbHMgaWQ9ImNhaXJvIj4KICAgICA8aWYgY29uZGl0aW9uLXNldD0ibGlu
dXgiPgpAQCAtNTI2LDggKzUyOCw5IEBACiAKICAgPGF1dG90b29scyBpZD0ibGlidnB4IgogICAg
IGF1dG9nZW4tdGVtcGxhdGU9IiUoc3JjZGlyKXMvY29uZmlndXJlIC0tcHJlZml4PSUocHJlZml4
KXMgLS1lbmFibGUtcGljIC0tYXM9eWFzbSAtLWRpc2FibGUtdW5pdC10ZXN0cyAtLXNpemUtbGlt
aXQ9MTYzODR4MTYzODQgLS1lbmFibGUtcG9zdHByb2MgLS1lbmFibGUtbXVsdGktcmVzLWVuY29k
aW5nIC0tZW5hYmxlLXRlbXBvcmFsLWRlbm9pc2luZyAtLWVuYWJsZS12cDktdGVtcG9yYWwtZGVu
b2lzaW5nIC0tZW5hYmxlLXZwOS1wb3N0cHJvYyAtLWVuYWJsZS1zaGFyZWQiPgotICAgIDxicmFu
Y2ggcmVwbz0iZ2l0aHViLmNvbSIgbW9kdWxlPSJ3ZWJtcHJvamVjdC9saWJ2cHguZ2l0IiB2ZXJz
aW9uPSIxLjYuMCIgY2hlY2tvdXRkaXI9ImxpYnZweC0xLjYuMCI+Ci0gICAgPC9icmFuY2g+Cisg
ICAgPGJyYW5jaCByZXBvPSJkb3dubG9hZHMud2VibXByb2plY3Qub3JnIgorICAgICAgICAgICAg
bW9kdWxlPSJsaWJ2cHgtMS42LjAudGFyLmJ6MiIgdmVyc2lvbj0iMS42LjAiCisgICAgICAgICAg
ICBoYXNoPSJzaGEyNTY6ZDBhZmJiNWViMWVjYWU2OGY4ZDU3OGFiYWNlMTYwYTk3ZTJlOGEyMzBl
MzAyOGNmNGRiMTE1ZDU5YTY5NWFhZCIvPgogICA8L2F1dG90b29scz4KIAogICAgPGF1dG90b29s
cyBpZD0iZ3N0LXBsdWdpbnMtb3BlbndlYnJ0YyIgc3VwcG9ydHMtcGFyYWxsZWwtYnVpbGRzPSJu
byIgc3VwcG9ydHMtbm9uLXNyY2Rpci1idWlsZHM9Im5vIiBhdXRvZ2VuLXNoPSIuL2F1dG9nZW4u
c2g7IC4vY29uZmlndXJlIj4K
</data>

          </attachment>
      

    </bug>

</bugzilla>