Skip to content
New issue

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

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

Already on GitHub? # to your account

Adding travis_wait to mvn install #3749

Closed
wants to merge 3 commits into from
Closed

Conversation

FanJups
Copy link

@FanJups FanJups commented Dec 23, 2019

Why ?
Building such big APIs as guava can take a lot of time. It's possible to get a time out.
Here, I got a time out due to generating javadoc. Then, I decided to add travis_wait and it works pretty well.

Official docs : https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received

Making my repository up-to-date
Why ?
Building such big APIs as guava   can take a lot of time.  It's possible to get a time out.
Here (4hwc/4HWCAutonomousCar@96c6396), I got a time out due to generating javadoc. 
Then, I decided to add travis_wait and it works pretty well.

Official docs : https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received
@cpovirk
Copy link
Member

cpovirk commented Dec 23, 2019

We have travis_wait on the script line below. As far as I know, we haven't seen similar problems with install. If that changes, we should make this change, but until then, let's keep it simple.

@cpovirk cpovirk closed this Dec 23, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants