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

Speed up Travis #3304

Merged
merged 3 commits into from
Dec 13, 2016
Merged

Speed up Travis #3304

merged 3 commits into from
Dec 13, 2016

Conversation

ickc
Copy link
Contributor

@ickc ickc commented Dec 12, 2016

  • travis use fast_finish
  • cabal uses -j

I added the comments on fast_finish into the file directly for future reference. This will gives the most "speed up" since it will returns the "green light" when the 2 allow_failure builds aren't finished (useful for pull request when waiting for the green light).

The cabal -j means it will set parallel jobs according to the number of CPUs. It only speed ups when there're multiple dependencies that can be installed at the same time. I noticed that gitit's travis build uses -j2 while it isn't used in pandoc. So may be there's a reason behind that?

@ickc
Copy link
Contributor Author

ickc commented Dec 13, 2016

Another possible "speed up" is to drop the cabal build with HEAD: I noticed that build almost always fails. It might just well be dropped to save travis' computer time (and to devote more thread to other builds by any of the projects under jgm).

@ickc
Copy link
Contributor Author

ickc commented Dec 13, 2016

Another possible drop: stack nightly osx, since stack nightly is already tested in Ubuntu. As far as testing "if it works for OS X" is concerned, the one stack 8.0.1 osx build already shows that.

As far as I know, Travis's OS X builds uses a completely different service, has 1 core only, and is not as reliable as the Ubuntu images. (more down time; and as seen in previous cases, "mysteriously" failed a build).

@jgm
Copy link
Owner

jgm commented Dec 13, 2016

I'd be okay with dropping the osx nightly build, if you want to add that to this PR.
I'd prefer to keep the HEAD build for now.

@ickc
Copy link
Contributor Author

ickc commented Dec 13, 2016

I remove the stack nightly in osx build by commenting it, so it is easier to get it back in the future if needed.

@jgm jgm merged commit 3354f5a into jgm:master Dec 13, 2016
@jgm
Copy link
Owner

jgm commented Dec 13, 2016

OK let's see how much this helps!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants