BUILD: Speed up Travis-CI builds #885

Merged
merged 1 commit into from Jan 5, 2017

Projects

None yet

2 participants

@csnover
Member
csnover commented Jan 5, 2017

container-based infrastructure (by sudo: false) starts and runs
faster:
https://docs.travis-ci.com/user/migrating-from-legacy/

gcc and make are preinstalled by Travis-CI for C++ language
containers, so do not need to be listed as dependencies:
https://docs.travis-ci.com/user/trusty-ci-environment/#Compilers-%26-Build-toolchain

ccache caching prevents wasting time rebuilding unchanged code:
https://docs.travis-ci.com/user/caching#ccache-cache

This should work OK, though there is a bug in travis lint which means that there is a slight chance this change might break something. I am happy to monitor and unbreak the build if that happens.

@csnover csnover BUILD: Speed up Travis-CI builds
container-based infrastructure (by sudo: false) starts and runs
faster:
https://docs.travis-ci.com/user/migrating-from-legacy/

gcc and make are preinstalled by Travis-CI for C++ language
containers, so do not need to be listed as dependencies:
https://docs.travis-ci.com/user/trusty-ci-environment/#Compilers-%26-Build-toolchain

ccache caching prevents wasting time rebuilding unchanged code:
https://docs.travis-ci.com/user/caching#ccache-cache
507562e
@sev-
Member
sev- commented Jan 5, 2017

Cool, Travis seems to be happy. Merging.

@sev- sev- merged commit 72f421c into scummvm:master Jan 5, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@csnover
Member
csnover commented Jan 6, 2017

Unfortunately looks like the compile time saved by ccache & quick start is perfectly balanced by the time it takes to send and receive the cache data. Travis say that they are working on backgrounding the cache transmission in the future (that is the slowest part), but right now it is treated as part of the job, so there is no decrease in build time. Plus, I’ve started seeing this bug, so perhaps this should be backed out for the moment and tried again later, at least once travis-ci/travis-ci#7103 is fixed.

@csnover csnover added a commit that referenced this pull request Jan 6, 2017
@csnover csnover Revert "Merge pull request #885 from csnover/faster-travis"
Travis builds are failing randomly after this change due to
upstream bug travis-ci/travis-ci#7103, so reverting these changes
until it is fixed.

This reverts commit 72f421c, reversing
changes made to c919c99.
0aba92b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment