Skip to content

feat: switch to ccache which is more maintained#1425

Draft
ankitm123 wants to merge 1 commit intojenkins-x:mainfrom
ankitm123:cache-issue
Draft

feat: switch to ccache which is more maintained#1425
ankitm123 wants to merge 1 commit intojenkins-x:mainfrom
ankitm123:cache-issue

Conversation

@ankitm123
Copy link
Copy Markdown
Member

@ankitm123 ankitm123 commented Feb 25, 2022

Signed-off-by: ankitm123 ankitmohapatra123@gmail.com

This is an experiment! Do not merge/approve.

Signed-off-by: ankitm123 <ankitmohapatra123@gmail.com>
@jenkins-x-bot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign ankitm123
You can assign the PR to them by writing /assign @ankitm123 in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


s.InRepoCache, err = lru.New(5000)
assert.NoError(t, err)
s.InRepoCache = lru.New(lru.Configure())
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

by default ccache sets the size to 5000.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants