Conversation
This reverts commit 61647a4.
|
Will this bring back the problem @achow101 reported in #12402 (comment) ?
|
|
@laanwj The problem exists in 3.4.1 (our current version) and is fixed in 3.4.2 (which is not yet released). I think we can also fix it by reverting to 3.3.4. |
|
Oops, sorry, I was confused there, then.
An alternative just mentioned @theuni was to remove ccache from the depends, and have it use the OS's ccache if installed and none if not. |
|
utACK faf8994 |
|
On second thought, this will cause travis to rebuild everything (all the depends) as everything depends on ccache, so better to merge only one change, and @fanquake is already working on a PR that will completely retire ccache. So better to hold off merging this for now. |
cc87967 depends: Remove ccache (fanquake) Pull request description: After discussion with @theuni, we can possibly just remove ccache from depends entirely. Related to #12606 Tree-SHA512: ae0a60c8d97467fa41d617daa48ed22159cf32613808634a983304901dd5ed27124e77868d2314004e5144f7b35ba1333f720bb12daec4c5ca03aaf29d593ef2
cc87967 depends: Remove ccache (fanquake) Pull request description: After discussion with @theuni, we can possibly just remove ccache from depends entirely. Related to bitcoin#12606 Tree-SHA512: ae0a60c8d97467fa41d617daa48ed22159cf32613808634a983304901dd5ed27124e77868d2314004e5144f7b35ba1333f720bb12daec4c5ca03aaf29d593ef2
According to #12515 cross-compiling for windows on linux requires a work-around. I believe that for this reason, the nightly builds for linux and windows are not available: https://bitcoin.jonasschnelli.ch/build/522
It should be safe to just revert to the previously known-to-be-working version of ccache.
Fixes #12515
Reverts one commit of #12402