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

Use travis native cache #6822

Merged
merged 1 commit into from Aug 3, 2016
Merged

Conversation

Kojoley
Copy link
Member

@Kojoley Kojoley commented Jul 24, 2016

- $HOME/.cache/pip
- $HOME/.cache/matplotlib
- $HOME/Library/Caches/pip
- ccache
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not clear from the travis docs that pip caches $HOME/Library/Caches/pip which is the pip cache directory on OSX.

@Kojoley
Copy link
Member Author

Kojoley commented Jul 24, 2016

As per travis-ci/travis-ci#4011 it is now support OSX too.

@jenshnielsen
Copy link
Member

I know that caching is supported on OSX that's why I added $HOME/Library/Caches/pip to the list of cached directories. But the documentation that you link to says that doing

cache:
  - pip

only adds

~/.cache/pip

to the cache and not $HOME/Library/Caches/pip

I can't see anything in the docs or the issue that you are linking to that indicates otherwise.

If you compare the OSX build of your PR to one of the Linux ones it seems to confirm that.
Linux:
https://s3.amazonaws.com/archive.travis-ci.org/jobs/146969225/log.txt
OSX:
https://api.travis-ci.org/jobs/146969231/log.txt?deansi=true

In the osx build all the dependencies are redownloaded.

@Kojoley
Copy link
Member Author

Kojoley commented Jul 24, 2016

Hmm. Even ccache was not used and stored on OSX

Edit: on Linux ccache does not work too (it requires language: c, see travis-ci/travis-ci#5853)

@Kojoley
Copy link
Member Author

Kojoley commented Jul 24, 2016

I have found that apt-get and brew currently is not cached. Should I reuse this PR or open a new one? (this seems can be done with cache: apt)

@jenshnielsen
Copy link
Member

Good catch. I think it's fine to use this one you can always change the title in the github web interface

@Kojoley
Copy link
Member Author

Kojoley commented Jul 24, 2016

Sigh. It is not possible now travis-ci/travis-ci#5876

@Kojoley Kojoley force-pushed the travis-native-cache branch 6 times, most recently from 777ccc5 to 5022018 Compare July 24, 2016 14:49
@Kojoley Kojoley force-pushed the travis-native-cache branch 4 times, most recently from 56d8336 to 121d0fc Compare July 24, 2016 17:20
@Kojoley
Copy link
Member Author

Kojoley commented Jul 24, 2016

Summary:

I have updated appveyor.yml with comments about results above.

@tacaswell tacaswell added this to the 2.1 (next point release) milestone Jul 26, 2016
@tacaswell tacaswell merged commit f078b24 into matplotlib:master Aug 3, 2016
@Kojoley Kojoley deleted the travis-native-cache branch January 5, 2020 21:17
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.

None yet

4 participants