Skip to content

WIP: Use ccache-toolchain#34

Closed
jakirkham wants to merge 3 commits intoconda-forge:v0.5from
jakirkham-feedstocks:use_ccache-toolchain
Closed

WIP: Use ccache-toolchain#34
jakirkham wants to merge 3 commits intoconda-forge:v0.5from
jakirkham-feedstocks:use_ccache-toolchain

Conversation

@jakirkham
Copy link
Copy Markdown
Member

Try using the ccache-toolchain to take advantage of ccache's compilation caching in the hopes of further speeding up this build.

This makes use of the regular toolchain and employs ccache to cache
previous compilations thus cutting down on build time. Should help speed
up our builds of Julia.
There shouldn't be any obvious changes from using ccache in the
compilation toolchain. That said, if there are, it would be good to be
able to distinguish and identify them. Hence bumping the build number to
get a new package.
@conda-forge-linter
Copy link
Copy Markdown

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@isuruf
Copy link
Copy Markdown
Member

isuruf commented Sep 19, 2017

You need to rerender using conda-forge/conda-smithy#526

@jakirkham
Copy link
Copy Markdown
Member Author

Do we get any benefits from using ccache without that re-rendering?

@isuruf
Copy link
Copy Markdown
Member

isuruf commented Sep 19, 2017

Not really

@jakirkham
Copy link
Copy Markdown
Member Author

Also should I be using ccache-toolchain or toolchain3?

@isuruf
Copy link
Copy Markdown
Member

isuruf commented Sep 19, 2017

ccache-toolchain is what's needed.

@isuruf
Copy link
Copy Markdown
Member

isuruf commented Sep 19, 2017

conda-forge/conda-smithy#526 is stalled because I didn't have time to test it. If you can test it here, that'd be great.

@bgruening bgruening closed this Aug 11, 2018
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.

4 participants