Sarcasm / compdb

The compilation database Swiss army knife
MIT License
292 stars 23 forks source link

Add header file to arguments #8

Closed nagua closed 6 years ago

nagua commented 6 years ago

I want to integrate compdb in my ci system for clang-tidy. I noticed, that clang-tidy is not able to run on a header file if the file is not listed inside the arguments of the generated compile arguments.

Sarcasm commented 6 years ago

Indeed, there is an issue, clang-tidy does not use the filename arguments, but use the file from the command line.

$ clang-tidy -p . Command.h
Error while processing /home/papin_g/dev/ws/irony/irony-mode/server/src/Command.h.

I would argue that maybe it's clang-tidy (or clang tools) that should be fixed, but still we can be accommodating.

Would you mind fixing the failing tests in the PR, so I can merge it?

nagua commented 6 years ago

Hey, the tests should be fixed now.

Sarcasm commented 6 years ago

Great! Thank you it's merged.