Workaround Travis-side CI issues by luke-jr · Pull Request #7487 · bitcoin/bitcoin

luke-jr · 2016-02-09T04:05:32Z

Currently Travis's wget fails fetching qrencode:

Fetching qrencode...
ERROR: no certificate subject alternative name matches
    requested host name `fukuchi.org'.
To connect to fukuchi.org insecurely, use `--no-check-certificate'.
OpenSSL: error:14077438:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert internal error
Unable to establish SSL connection.
make: *** [/home/travis/build/luke-jr/bitcoin/depends/sources/download-stamps/.stamp_fetched-qrencode-qrencode-3.4.4.tar.bz2.hash] Error 4

Currently Travis's wget fails fetching qrencode: Fetching qrencode... ERROR: no certificate subject alternative name matches requested host name `fukuchi.org'. To connect to fukuchi.org insecurely, use `--no-check-certificate'. OpenSSL: error:14077438:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert internal error Unable to establish SSL connection. make: *** [/home/travis/build/luke-jr/bitcoin/depends/sources/download-stamps/.stamp_fetched-qrencode-qrencode-3.4.4.tar.bz2.hash] Error 4

laanwj · 2016-02-09T07:56:16Z

What wonders me most is why it won't try the fallback URL

luke-jr · 2016-02-09T08:12:23Z

Not sure, but even if it did, it would fail. Note the second commit I had to add for the OSX SDK on bitcoincore.org :/

laanwj · 2016-02-09T08:23:39Z

Why is the MacOSX SDK download broken too?

I don't think I understand what is the issue here. Is that site irrevocably broken, or is there some certificate issue? (e.g. missing ca-certificates package?) But in that case you'd expect any tool to fail.

Switching to another tool is possible but I'd like to know what is wrong first. It seems like a heavy-handed solution.

@theuni any idea?

luke-jr · 2016-02-09T08:25:50Z

Seems to be a wget bug in the ancient 12.02 Ubuntu that Travis uses. But I could be wrong.

laanwj · 2016-02-09T09:22:18Z

Bleh. We need 14.04 support for other reasons too (e.g. c++11).

luke-jr · 2016-02-09T09:25:01Z

It looks like 14.04 is possible, but... only be losing IPv6 localhost support [again]. :/

dcousens · 2016-02-09T11:52:41Z

concept ACK

luke-jr · 2016-02-09T20:10:58Z

Note: curl needs -L to follow redirects.

…[on HTTP error response] with curl

luke-jr · 2016-02-10T02:19:24Z

This fixes the Travis-side CI issues: https://travis-ci.org/luke-jr/bitcoin/builds/108146026

btcdrak · 2016-02-10T09:19:43Z

utACK, agree on using curl

maflcko · 2016-02-13T16:09:54Z

depends/builders/linux.mk

@@ -1,2 +1,2 @@
 build_linux_SHA256SUM = sha256sum
-build_linux_DOWNLOAD = wget --timeout=$(DOWNLOAD_CONNECT_TIMEOUT) --tries=$(DOWNLOAD_RETRIES) -nv -O
+build_linux_DOWNLOAD = curl --location --fail --connect-timeout $(DOWNLOAD_CONNECT_TIMEOUT) --retry $(DOWNLOAD_RETRIES) -L -o


Is it required to set --location and -L?

maflcko · 2016-02-13T16:11:41Z

utACK c01f08d

149641e Travis: Use Blue Box VMs for IPv6 loopback support (Luke Dashjr) c01f08d Bugfix: depends/Travis: Use --location (follow redirects) and --fail [on HTTP error response] with curl (Luke Dashjr) 5d1148c Travis: Use curl rather than wget for Mac SDK (Luke Dashjr) 1ecbb3b depends: Use curl for fetching on Linux (Luke Dashjr)

Github-Pull: bitcoin#7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b

Github-Pull: #7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b

Github-Pull: bitcoin#7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b

Github-Pull: bitcoin#7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b --- Removed sudo, dist and group, because we are on a different box.

Github-Pull: bitcoin#7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b Cherry-pick-From: 00d57b4

* Fix url in .travis.yml Github-Pull: bitcoin#7136 Rebased-From: 9999cb0 * Workaround Travis-side CI issues Github-Pull: bitcoin#7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b * [depends] builders: No need to set -L and --location for curl Github-Pull: bitcoin#7606 Rebased-From: fa7a5c5 * Bugfix: gitian: Add curl to packages (now needed for depends) Conflicts: contrib/gitian-descriptors/gitian-osx.yml Github-Pull: bitcoin#7614 Rebased-From: 5c70a6d

Darwin build fixes Includes fixes cherry-picked from the following upstream PRs: - bitcoin/bitcoin#7136 - Only the third commit (to avoid a merge conflict) - bitcoin/bitcoin#7302 - Excluding the first commit, which is unnecessary (we use Boost 1.62) - bitcoin/bitcoin#7487 - bitcoin/bitcoin#7606 - bitcoin/bitcoin#7711 - bitcoin/bitcoin#7165 - bitcoin/bitcoin#8002 - bitcoin/bitcoin#8210 - Only the second commit - bitcoin/bitcoin#9114

Github-Pull: bitcoin#7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b (cherry picked from commit 6bf4884)

luke-jr added 2 commits February 9, 2016 04:02

Travis: Use curl rather than wget for Mac SDK

5d1148c

laanwj added Build system Tests labels Feb 9, 2016

luke-jr added 2 commits February 9, 2016 22:15

Bugfix: depends/Travis: Use --location (follow redirects) and --fail …

c01f08d

…[on HTTP error response] with curl

Travis: Use Blue Box VMs for IPv6 loopback support

149641e

luke-jr changed the title ~~depends: Use curl for fetching on Linux~~ Workaround Travis-side CI issues Feb 10, 2016

maflcko reviewed Feb 13, 2016
View reviewed changes

laanwj merged commit 149641e into bitcoin:master Feb 15, 2016

maflcko pushed a commit to maflcko/bitcoin-core that referenced this pull request Feb 26, 2016

Workaround Travis-side CI issues

6bf4884

Github-Pull: bitcoin#7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b

laanwj pushed a commit that referenced this pull request Mar 1, 2016

Workaround Travis-side CI issues

00d57b4

Github-Pull: #7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b

laanwj pushed a commit that referenced this pull request Mar 1, 2016

Workaround Travis-side CI issues

77841d4

Github-Pull: #7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b

schinzelh pushed a commit to dashpay/dash that referenced this pull request Mar 7, 2016

Workaround Travis-side CI issues

3c36bcf

Github-Pull: bitcoin#7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b

dexX7 pushed a commit to dexX7/bitcoin that referenced this pull request Apr 24, 2016

Workaround Travis-side CI issues

5a8b92b

Github-Pull: bitcoin#7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b --- Removed sudo, dist and group, because we are on a different box.

braydonf pushed a commit to braydonf/bitcoin that referenced this pull request May 12, 2016

Workaround Travis-side CI issues

79b6425

Github-Pull: bitcoin#7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b Cherry-pick-From: 00d57b4

braydonf pushed a commit to braydonf/bitcoin that referenced this pull request Jun 2, 2016

Workaround Travis-side CI issues

128c5e1

Github-Pull: bitcoin#7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b Cherry-pick-From: 00d57b4

luke-jr added a commit to bitcoinknots/bitcoin that referenced this pull request Jun 18, 2016

Null-Merge bitcoin#7487 depends_curl: Already in v0.12.1

536812c

braydonf pushed a commit to braydonf/bitcoin that referenced this pull request Oct 4, 2016

Workaround Travis-side CI issues

98a2f26

Github-Pull: bitcoin#7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b Cherry-pick-From: 00d57b4

langerhans mentioned this pull request Dec 11, 2016

[depends] fix multiple depends url issues dogecoin/dogecoin#1358

Merged

str4d mentioned this pull request Oct 29, 2017

Darwin build fixes zcash/zcash#2697

Merged

laanwj mentioned this pull request Nov 20, 2018

travis: Do not specify sudo in .travis #14770

Merged

reddink pushed a commit to reddcoin-project/reddcoin-3.10 that referenced this pull request May 27, 2020

Workaround Travis-side CI issues

ecc0c6e

Github-Pull: bitcoin#7487 Rebased-From: 149641e c01f08d 5d1148c 1ecbb3b (cherry picked from commit 6bf4884)

bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Workaround Travis-side CI issues#7487

Workaround Travis-side CI issues#7487
laanwj merged 4 commits intobitcoin:masterfrom
luke-jr:depends_curl

luke-jr commented Feb 9, 2016

Uh oh!

laanwj commented Feb 9, 2016

Uh oh!

luke-jr commented Feb 9, 2016

Uh oh!

laanwj commented Feb 9, 2016

Uh oh!

luke-jr commented Feb 9, 2016

Uh oh!

laanwj commented Feb 9, 2016

Uh oh!

luke-jr commented Feb 9, 2016

Uh oh!

dcousens commented Feb 9, 2016

Uh oh!

luke-jr commented Feb 9, 2016

Uh oh!

luke-jr commented Feb 10, 2016

Uh oh!

btcdrak commented Feb 10, 2016

Uh oh!

maflcko Feb 13, 2016

Uh oh!

maflcko commented Feb 13, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Conversation

luke-jr commented Feb 9, 2016

Uh oh!

laanwj commented Feb 9, 2016

Uh oh!

luke-jr commented Feb 9, 2016

Uh oh!

laanwj commented Feb 9, 2016

Uh oh!

luke-jr commented Feb 9, 2016

Uh oh!

laanwj commented Feb 9, 2016

Uh oh!

luke-jr commented Feb 9, 2016

Uh oh!

dcousens commented Feb 9, 2016

Uh oh!

luke-jr commented Feb 9, 2016

Uh oh!

luke-jr commented Feb 10, 2016

Uh oh!

btcdrak commented Feb 10, 2016

Uh oh!

maflcko Feb 13, 2016

Choose a reason for hiding this comment

Uh oh!

maflcko commented Feb 13, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants