Skip to content

[0.16] depends: Switch to downloading expat from GitHub#12585

Merged
maflcko merged 1 commit intobitcoin:0.16from
fanquake:0-16-expat
Mar 5, 2018
Merged

[0.16] depends: Switch to downloading expat from GitHub#12585
maflcko merged 1 commit intobitcoin:0.16from
fanquake:0-16-expat

Conversation

@fanquake
Copy link
Member

@fanquake fanquake commented Mar 3, 2018

@droark mentioned the expat download was failing when trying to gitian build 0.16.
I tried locally and also saw issues:

make download
Checksum missing or mismatched for expat source. Forcing re-download.
Fetching expat-2.2.1.tar.bz2 from https://downloads.sourceforge.net/project/expat/expat/2.2.1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:--  0:00:01 --:--:--     0
100   652  100   652    0     0    361      0  0:00:01  0:00:01 --:--:--     0
/Users/xxx/Github/bitcoin/depends/work/download/expat-2.2.1/expat-2.2.1.tar.bz2.temp: FAILED
shasum: WARNING: 1 computed checksum did NOT match
Fetching expat-2.2.1.tar.bz2 from https://bitcoincore.org/depends-sources
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:--  0:00:01 --:--:--     0
curl: (22) The requested URL returned error: 404 Not Found
make[1]: *** [/Users/xxx/Github/bitcoin/depends/sources/download-stamps/.stamp_fetched-expat-expat-2.2.1.tar.bz2.hash] Error 22
make: *** [download-linux] Error 2

This switches to downloading from GitHub, as we are doing in master.

@maflcko
Copy link
Member

maflcko commented Mar 3, 2018

utACK 25afebc

@droark
Copy link
Contributor

droark commented Mar 4, 2018

utACK

Am busy right now but will try to convert this to a tACK in the next couple of days.

@practicalswift
Copy link
Contributor

utACK 25afebc

@maflcko maflcko merged commit 25afebc into bitcoin:0.16 Mar 5, 2018
maflcko pushed a commit that referenced this pull request Mar 5, 2018
25afebc depends: Switch expat download to GitHub (fanquake)

Pull request description:

  @droark [mentioned](#12402 (comment)) the expat download was failing when trying to gitian build 0.16.
  I tried locally and also saw issues:
  ```
  make download
  Checksum missing or mismatched for expat source. Forcing re-download.
  Fetching expat-2.2.1.tar.bz2 from https://downloads.sourceforge.net/project/expat/expat/2.2.1
    % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                   Dload  Upload   Total   Spent    Left  Speed
    0     0    0     0    0     0      0      0 --:--:--  0:00:01 --:--:--     0
  100   652  100   652    0     0    361      0  0:00:01  0:00:01 --:--:--     0
  /Users/xxx/Github/bitcoin/depends/work/download/expat-2.2.1/expat-2.2.1.tar.bz2.temp: FAILED
  shasum: WARNING: 1 computed checksum did NOT match
  Fetching expat-2.2.1.tar.bz2 from https://bitcoincore.org/depends-sources
    % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                   Dload  Upload   Total   Spent    Left  Speed
    0     0    0     0    0     0      0      0 --:--:--  0:00:01 --:--:--     0
  curl: (22) The requested URL returned error: 404 Not Found
  make[1]: *** [/Users/xxx/Github/bitcoin/depends/sources/download-stamps/.stamp_fetched-expat-expat-2.2.1.tar.bz2.hash] Error 22
  make: *** [download-linux] Error 2
  ```
  This switches to downloading from GitHub, as we are doing in master.

Tree-SHA512: 0e3072ce4d8fee3e2bc434afc7469968c21e01ca35aa2ccbffc25f8e9b00af2e4a5f3f9e4cd5c0cd8e72a10b16205236eb0536380849109a739c09358dbc1394
@fanquake fanquake deleted the 0-16-expat branch April 12, 2018 02:05
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants