uwreact / frc_control

ROS metapackage enabling FRC teams to use ROS on their robots
BSD 3-Clause "New" or "Revised" License
5 stars 0 forks source link

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

Open matthew-reynolds opened 5 years ago

matthew-reynolds commented 5 years ago

🐛 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.