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

clang-tidy: Add clang-tidy executable path argument #494

Open
wants to merge 5 commits into
base: humble
Choose a base branch
from

Conversation

jjannik
Copy link

@jjannik jjannik commented May 29, 2024

This MR allows an easy way to add clang-tidy caching
like here https://github.com/matus-chochlik/ctcache or https://github.com/ejfitzgerald/clang-tidy-cache/

Copy link
Contributor

@clalancette clalancette left a comment

Choose a reason for hiding this comment

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

I guess my question here is: what is the use case? Because it seems like we can already influence which clang-tidy binary is called by changing the PATH environment variable, so why do we need this?

@jjannik
Copy link
Author

jjannik commented Jun 13, 2024

True. As far as I can see you can not really influence the name of your executable. This would allow you to call something like cache-clang-tidy or whatever is expected for the above caching solutions. Which would make it clear to the caller that not actually clang tidy is used, more like a wrapper.

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.

2 participants