[travis] Exit early when check-doc.py fails #7455

1. 
   MarcoFalke commented at 3:38 pm on February 1, 2016: member

   Discussed on IRC
2. 
   laanwj commented at 3:41 pm on February 1, 2016: member

   Concept ACK.

   Can we test this by making a dummy change and see if travis fails, and how it looks now?

3. laanwj added the label Tests on Feb 1, 2016
4. MarcoFalke force-pushed on Feb 1, 2016
5. MarcoFalke force-pushed on Feb 1, 2016
6. MarcoFalke force-pushed on Feb 1, 2016
7. MarcoFalke force-pushed on Feb 1, 2016
8. MarcoFalke force-pushed on Feb 1, 2016
9. 
   MarcoFalke commented at 8:03 pm on February 1, 2016: member

   dummy change: https://travis-ci.org/bitcoin/bitcoin/builds/106296202

   TBH, I don’t like the set -e workaround, as the travis build will not exit cleanly. It doesn’t matter with the current .travis.yml but it may in the future and it is not considered best practice.

10. 
    laanwj commented at 11:28 am on February 2, 2016: member

    @theuni any ideas here? We want to terminate the test prematurely if the doc check fails, to avoid wasting resources, is there a nicer way to do this then set -e or exit?
11. 
    MarcoFalke commented at 12:46 pm on February 2, 2016: member

    @laanwj You could “remember” the exit code and branch subsequent statements/lines but this is a somewhat larger diff, I guess.
12. 
    MarcoFalke commented at 10:02 am on February 26, 2016: member

    @laanwj The first commit should be uncontroversial and improve things considerably without making anything worse. Would you mind if I create a new pull for this commit?
13. 
    laanwj commented at 2:22 pm on March 2, 2016: member

    As this is a testing/build system related pull, I’d just like @theuni to take a look at it. If he’s ok with it we’ll just merge.
14. 
    theuni commented at 1:31 am on March 3, 2016: member

    @MarcoFalke I think it’d do what you wanted if you just bumped it up into before_script. Then you wouldn’t have to worry about forcing an exit, as before_script errors the build on any non-zero return.
15. 
    theuni commented at 1:32 am on March 3, 2016: member

    other than that, ACK.
16. [travis] Exit early when check-doc.py fails fa5f19319a
17. MarcoFalke force-pushed on Mar 3, 2016
18. MarcoFalke force-pushed on Mar 3, 2016
19. 
    MarcoFalke commented at 3:23 pm on March 3, 2016: member

    Thanks for letting me know. This is indeed working: https://travis-ci.org/bitcoin/bitcoin/jobs/113383026
20. laanwj merged this on Mar 3, 2016
21. laanwj closed this on Mar 3, 2016

    ---

22. laanwj referenced this in commit 7f966713a4 on Mar 3, 2016
23. MarcoFalke deleted the branch on Mar 3, 2016
24. codablock referenced this in commit 6ca515e4e9 on Sep 16, 2017
25. codablock referenced this in commit 8716aa04b5 on Sep 19, 2017
26. codablock referenced this in commit f9777b8c68 on Dec 9, 2017
27. codablock referenced this in commit 6f68ec6fe7 on Dec 9, 2017
28. codablock referenced this in commit 53ec957122 on Dec 9, 2017
29. codablock referenced this in commit 48eddeea29 on Dec 11, 2017
30. MarcoFalke locked this on Sep 8, 2021

Contributors
MarcoFalke laanwj theuni

Labels
Tests