2 quick changes:
- retry tests if they fail to help work-around false-negatives. Closes #4923
- Use DEBUG_LOCKORDER for one test
There's also a little fixup for depends necessary to get the flags setup correctly.
Travis tweaks #4932
pull theuni wants to merge 3 commits into bitcoin:master from theuni:travis changing 2 files +3 −3-
theuni commented at 6:39 PM on September 16, 2014: member
-
travis: retry tests before giving up 7e3821c097
-
depends: respect CPPFLAGS when configuring with depends cf12c223d0
-
travis: use DEBUG_LOCKORDER for our quick/small Linux build 1b0c981cb6
-
BitcoinPullTester commented at 6:53 PM on September 16, 2014: none
Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4932_1b0c981cb69c63f09fef71241ded8a6279e4b993/ for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
-
sipa commented at 6:53 PM on September 16, 2014: member
How many times will it retry?
-
theuni commented at 7:15 PM on September 16, 2014: member
- Docs here: http://docs.travis-ci.com/user/build-timeouts/ .
For a data point, the tests run for about 2min when successful.
-
theuni commented at 7:16 PM on September 16, 2014: member
Heh, github munged my response. It retries three (t-h-r-e-e) times. Silly markdown.
-
sipa commented at 7:19 PM on September 16, 2014: member
ACK
- gavinandresen referenced this in commit d65ea7efa3 on Sep 16, 2014
- gavinandresen merged this on Sep 16, 2014
- gavinandresen closed this on Sep 16, 2014
- MarcoFalke locked this on Sep 8, 2021
Contributors