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

Remove ccache from depends subsystem #3187

Merged
merged 2 commits into from
Nov 4, 2019

Conversation

codablock
Copy link

This backports bitcoin#12607. The reason to backport it now is different then the reason it was initially removed in Bitcoin. In Bitcoin is was removed due to build failures related to cross compliation of Windows. In Dash I'd like to remove it because Gitlab is currently failing to download ccache due to an expired https certificate (https://samba.org).

After this PR it will be required to have ccache installed on your system, same as with having gcc pre-installed before being able to build.

MarcoFalke and others added 2 commits November 4, 2019 13:20
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
@UdjinM6 UdjinM6 added this to the 14.1 milestone Nov 4, 2019
Copy link

@nmarley nmarley left a comment

Choose a reason for hiding this comment

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

👍Been waiting for this, didn't make sense to me to have this in depends.

tested ACK

Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

ACK (tested on local mac)

@UdjinM6 UdjinM6 merged commit ff1869d into dashpay:develop Nov 4, 2019
@codablock codablock deleted the pr_remove_ccache branch November 5, 2019 08:16
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.

3 participants