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

Travis: fix cache dir #1164

Merged
merged 2 commits into from Sep 15, 2017

Conversation

Projects
None yet
2 participants
@Slamdunk
Contributor

Slamdunk commented Sep 15, 2017

On previous PR I specified a cache dir for composer download packages according to common standards $HOME/.composer, but it resulted never working here.

After deep investigation I found out that composer is a bit fancy about how to decide cache dir, and the reliable way to cache dependencies is to set cache dir to $HOME/.composer/cache.
The cache now works, saving us ~20 seconds per build (and a lot of external expensive calls).

A second change is in place: on Travis be verbose about tests so we can track easily skipped/incomplete/risky tests.

@Synchro Synchro merged commit 5494306 into PHPMailer:master Sep 15, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Synchro

This comment has been minimized.

Member

Synchro commented Sep 15, 2017

Thanks

@Slamdunk Slamdunk deleted the Slamdunk:travis_fix_cache branch Sep 15, 2017

@Slamdunk

This comment has been minimized.

Contributor

Slamdunk commented Sep 15, 2017

Of course you'll see the cache working only on commits after this merge, since the merge build is the first that will create the cache.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment