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.

Sign up for GitHub

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

Jump to bottom

Enable clang-tidy on cross-compilation-exclusive files #83

Open
matthew-reynolds opened this issue Jul 30, 2019 · 0 comments
Open

Enable clang-tidy on cross-compilation-exclusive files #83

matthew-reynolds opened this issue Jul 30, 2019 · 0 comments
Labels
🐛 bug 🆘 help wanted

Comments

@matthew-reynolds
Copy link
Member

🐛 Bug Report

clang-tidy relies on the compile_commands.json files outputted by cmake during compilation. Unfortunately, this means that

a) code must always be compiled before running clang-tidy
b) code that is not compiled (For example, files specific to real HW) are not analyzed

In addition, simply cross-compiling and then running the clang-tidy script causes the tool to get confused by the cross-compilation and report tons of irrelevant errors as well as misreporting true errors.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🐛 bug 🆘 help wanted
Milestone
No milestone
Development

No branches or pull requests
1 participant
@matthew-reynolds