New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prefix `apt-get -qq with `travis_wait` #442

Merged
merged 1 commit into from Aug 4, 2015

Conversation

Projects
None yet
2 participants
@pquentin
Contributor

pquentin commented Aug 3, 2015

apt-get -qq is completely quiet. If apt-get takes more than 10
minutes to execute, nothing will be outputted for 10 minutes, and travis
will kill the build. Using travis_wait ensures the commands will not be
killed for 20 minutes. See the documentation for details.

Prefix `apt-get -qq with `travis_wait`
`apt-get -qq` is completely quiet. If `apt-get` takes more than 10
minutes to execute, nothing will be outputted for 10 minutes, and travis
will kill the build. Using `travis_wait` ensures the commands will not be
killed for 20 minutes. See [the documentation] for details.

[the documentation]: http://docs.travis-ci.com/user/build-timeouts/#Build-times-out-because-no-output-was-received)
@pquentin

This comment has been minimized.

Show comment
Hide comment
@pquentin

pquentin Aug 4, 2015

Contributor

This appears to fix the issue: all tests passed with travis_wait. (An earlier version of the pull request used it incorrectly, it's travis_wait sudo ..., not sudo travis_wait ....)

I was told that iceiceice should be assigned, but I don't know his GitHub handle and cannot assign myself.

Contributor

pquentin commented Aug 4, 2015

This appears to fix the issue: all tests passed with travis_wait. (An earlier version of the pull request used it incorrectly, it's travis_wait sudo ..., not sudo travis_wait ....)

I was told that iceiceice should be assigned, but I don't know his GitHub handle and cannot assign myself.

aquileia added a commit that referenced this pull request Aug 4, 2015

Merge pull request #442 from pquentin/travis-wait
Prefix `apt-get -qq  with `travis_wait` to fix timeout

@aquileia aquileia merged commit f3ee454 into wesnoth:master Aug 4, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment