Skip to content
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

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants