Build: add REPO_SLUG to fix errors in forked builds #17418

pull RandyMcMillan wants to merge 1 commits into bitcoin:master from RandyMcMillan:repo-slug changing 2 files +4 −2
  1. RandyMcMillan commented at 2:24 PM on November 8, 2019: contributor

    This small change enables forks to use the TravisCI tests.

  2. Build: add REPO_SLUG to fix errors in forked builds
    This eliminates error on forked repos in TravisCI instances
    696b951556
  3. fanquake added the label Tests on Nov 8, 2019
  4. fanquake requested review from MarcoFalke on Nov 8, 2019
  5. RandyMcMillan commented at 4:32 PM on November 8, 2019: contributor

    @hebasto - I am not sure if eliminating the if condition all together is a possibility.

  6. MarcoFalke commented at 4:35 PM on November 8, 2019: member

    What kind of errors?

  7. RandyMcMillan commented at 5:19 PM on November 8, 2019: contributor

    What kind of errors?

    I believe this is an example. https://travis-ci.org/RandyMcMillan/bitcoin/jobs/608862338#L17505 https://travis-ci.org/RandyMcMillan/bitcoin/jobs/608862338#L17515

    It seems the if condition is timing out builds. I could be missing something but this change seems have fixed it.

    NOTE: If people are using a local environment then the $TRAVIS_REPO_SLUG will do the same thing. SoI believe it is best to make the logic compare an environment variable that is set up.

  8. MarcoFalke commented at 5:26 PM on November 8, 2019: member

    The error comes from ./ci/test/06_script_a.sh. Here you are modifying linters, so this can not be the right "fix".

  9. MarcoFalke closed this on Nov 8, 2019

  10. MarcoFalke commented at 5:27 PM on November 8, 2019: member

    The error is

    Error! Initial build successful, but not enough time remains to run later build stages and tests. Please manually re-run this job by using the travis restart button or asking a bitcoin maintainer to restart. The next run should not time out because the build cache has been saved.

    Which means that your repository has a limit of 50 minutes. There is no code change that could fix that. You'd have to ask travis to increase it.

  11. RandyMcMillan deleted the branch on Nov 8, 2019
  12. RandyMcMillan commented at 7:00 PM on November 8, 2019: contributor

    Thanks - lesson learned - At least something useful came out of it.

  13. MarcoFalke referenced this in commit a8d9f7dfa7 on Nov 20, 2019
  14. sidhujag referenced this in commit 42caf11783 on Nov 20, 2019
  15. sidhujag referenced this in commit f5ef254714 on Nov 10, 2020
  16. Munkybooty referenced this in commit 1365fd54e1 on Jun 7, 2022
  17. Munkybooty referenced this in commit b2ea31d605 on Jun 8, 2022
  18. Munkybooty referenced this in commit 9f9c3dc725 on Jun 16, 2022
  19. DrahtBot locked this on Aug 16, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-17 06:14 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me