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

Add ability to include llvm tools (clang-tidy/format) #238

Closed
acmnpv opened this issue Mar 16, 2020 · 2 comments · Fixed by #244
Closed

Add ability to include llvm tools (clang-tidy/format) #238

acmnpv opened this issue Mar 16, 2020 · 2 comments · Fixed by #244

Comments

@acmnpv
Copy link

acmnpv commented Mar 16, 2020

Currently, adding the llvm building block only adds the base compiler to the image.
It would be great if it was also possible to add the corresponding tools as well, to allow the generated image to be used for source code linting.

@samcmill
Copy link
Collaborator

Yes, I can add clang-format and clang-tidy as an optional install.

It looks like the CentOS distro packages already include clang-format with the compiler.

@acmnpv
Copy link
Author

acmnpv commented Mar 20, 2020

Great!
I have worked around this for now by adding the install manually, but having this as an option in the script will be a cleaner solution.

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 a pull request may close this issue.

2 participants