Fix up ccache usage on Travis + try enabling on OSX #7651
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I was still going on the PR, I just closed it to prevent travisCI pollution and spurious emails to devs. Anyway, from my latest test run on https://travis-ci.org/andyfaff/scipy/builds/256840423 I see a reduction from a 15min build ( |
pv
force-pushed
the
travis-ccache-2
branch
4 times, most recently
from
e4407f5
to
ab74326
Compare
Change the USE_SDIST and USE_WHEEL targets to do builds in a more ccache-friendly way. Also add a script tools/suppress_output.py for suppressing commands outputs and printing tickers.
|
Ok, it's not using ccache on osx with my config. Probably best to just
copypaste yours...
|
|
Ok, seems to work now. |
|
Do you see speedup in Linux builds? |
|
ccache -s shows about 300 cache hits from the build, so it works, both on
linux and osx. Osx has only few cache misses, but the linux builds had 80
or so. This is regardless of wheel/sdist so it's not related to this PR.
The recent precise to trusty transition seems to have changed something.
Unclear what.
26.7.2017 0.31 "Andrew Nelson" <notifications@github.com> kirjoitti:
… Do you see speedup in Linux builds?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#7651 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AACI5mO4YQXZaiH15mpF2lIbs3jgNkD_ks5sRmy9gaJpZM4OhvHs>
.
|
|
I looked through the changes to |
|
Thanks for the help with this @andyfaff. |
|
I'm very impressed with the speed of the OS X build/test. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix up ccache usage on USE_SDIST and USE_WHEEL targets.
Pip by default builds in a directory with random name, which throws off the compiler result caching.
Also try to enable the caching on OSX, based on gh-7647 which was quite close.
Finally, add and use some tools for suppressing the huge output from scipy build commands
in the USE_SDIST and USE_WHEEL targets.