[travis] Print the commit which was evaluated #7798

pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:Mf1604-travisCommitLog changing 1 files +3 −0

MarcoFalke commented at 12:32 PM on April 3, 2016: member

The travis test result which is linked to by most commits on GitHub is sometimes not related to the commit.

It may happen that even the build jobs of a single build contains results from different commits.
[travis] Print the commit which was evaluated faef3b710f
MarcoFalke force-pushed on Apr 3, 2016
MarcoFalke force-pushed on Apr 3, 2016
laanwj added the label Tests on Apr 3, 2016
jonasschnelli commented at 7:24 AM on April 5, 2016: contributor

Travis error seems unrelated. Just re-kicked travis for this PR.
theuni commented at 7:40 AM on April 5, 2016: member

Maybe you'd like to print $TRAVIS_COMMIT_RANGE, so that it's immediately clear what was in the build? I believe it should print "foo...bar", which could be easily checked locally.
MarcoFalke commented at 10:31 AM on April 5, 2016: member
Right now it should print
```
commit $some_random_hash Merge: 30c2dd8 faef3b7
```
where 30c2dd8 should be in bitcoin/master and faef3b7 in the branch someone wants to merge. I think that is sufficient to quickly check that something went wrong.
MarcoFalke commented at 10:37 AM on April 5, 2016: member
Travis error seems unrelated

Travis is building random crap, which you can see by observing the commit ids:
```
$ echo $TRAVIS_COMMIT_LOG
commit 73c560fc0d92177e88ddd02f6ac628340b85688c Merge: 30c2dd8 4b1ea9a
```
where 30c2dd8 is the current HEAD and 4b1ea9a is some random commit.

So the travis error is "related" but unwanted.
[travis] echo $TRAVIS_COMMIT_RANGE fac9ca2ec6

MarcoFalke commented at 11:02 AM on April 5, 2016: member

@theuni Is it ok if I print both, so it is clear which commit of the target branch travis used? Also, I'd rather not trust $TRAVIS_COMMIT_RANGE blindly.

$ echo $TRAVIS_COMMIT_RANGE
30c2dd8d0598afac9319d3a813a89a682b9c0d34...fac9ca2ec69bc629dbbd81c83e5bf67690e52c96
$ echo $TRAVIS_COMMIT_LOG
commit 049d5e4dfe711bb2af5c9c3904de7f78a24fedac Merge: a914968 fac9ca2

MarcoFalke commented at 11:37 AM on April 9, 2016: member

Last travis failure indeed unrelated:

Failed to fetch http://us.archive.ubuntu.com/ubuntu/pool/main/g/gcc-4.6/libstdc++6_4.6.3-1ubuntu5_i386.deb  Unable to connect to us.archive.ubuntu.com:http: [IP: 2001:67c:1562::15 80]

Anything holding this back?

sipa commented at 11:40 AM on April 9, 2016: member

Restarting travis.
sipa commented at 3:39 PM on April 12, 2016: member

utACK
theuni commented at 4:33 PM on April 12, 2016: member

utACK https://github.com/bitcoin/bitcoin/pull/7798/commits/fac9ca2ec69bc629dbbd81c83e5bf67690e52c96
laanwj merged this on Apr 15, 2016
laanwj closed this on Apr 15, 2016
laanwj referenced this in commit cabba24a5b on Apr 15, 2016
MarcoFalke deleted the branch on Apr 15, 2016
codablock referenced this in commit d24080d8f9 on Sep 16, 2017
codablock referenced this in commit cf40170620 on Sep 19, 2017
codablock referenced this in commit 60159eb2f4 on Dec 20, 2017
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels