Skip to content
This repository has been archived by the owner on Aug 16, 2022. It is now read-only.

feat(ansible): add ccache role #322

Merged
merged 2 commits into from
May 21, 2022
Merged

feat(ansible): add ccache role #322

merged 2 commits into from
May 21, 2022

Conversation

brkay54
Copy link
Member

@brkay54 brkay54 commented May 21, 2022

Signed-off-by: Berkay [email protected]

Description

Closes ( autowarefoundation/autoware#2442).

In autoware documentation page, Ccache is recommended to speed up recompilation. See here.

The purpose of this PR is adding Ccache into docker images.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@brkay54 brkay54 self-assigned this May 21, 2022
@brkay54 brkay54 requested a review from kenji-miyake May 21, 2022 18:11
kenji-miyake
kenji-miyake previously approved these changes May 21, 2022
@kenji-miyake kenji-miyake enabled auto-merge (squash) May 21, 2022 18:18
Co-authored-by: Kenji Miyake <[email protected]>
Signed-off-by: Berkay <[email protected]>
auto-merge was automatically disabled May 21, 2022 18:18

Head branch was pushed to by a user without write access

@kenji-miyake kenji-miyake enabled auto-merge (squash) May 21, 2022 18:19
@kenji-miyake
Copy link
Contributor

Please let me edit the title because it's not only for Docker. 🙇

@kenji-miyake kenji-miyake changed the title feat(ansible): add ccache into docker image feat(ansible): add ccache role May 21, 2022
@kenji-miyake kenji-miyake enabled auto-merge (squash) May 21, 2022 18:20
@kenji-miyake
Copy link
Contributor

@brkay54 As I'm working on improvement around Humble Docker support, please understand that it might take a bit of time to reflect this or that the docker image name might change. 🙇

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants