Oh, I see. In that case yea, I think just "don't run clang-tidy on those .cc files" is a good bet. But you'd still probably want to run tidy on other .cc files that include headers in that dir, and that's where the regex comes in handy.
On Tue, Jul 18, 2017 at 7:59 PM, Henry Robinson <[email protected]> wrote: > Does this exclude source files? The issue I have is that I want to exclude > a whole directory of .cc files. > > On 18 July 2017 at 17:09, Todd Lipcon <[email protected]> wrote: > > > FYI I have a patch up against clang-tidy to add this feature: > > https://reviews.llvm.org/D34654 > > Hoping it will be reviewed/committed soon. > > > > -Todd > > > > On Thu, Jul 13, 2017 at 10:58 AM, Henry Robinson <[email protected]> > > wrote: > > > > > To close this loop - I took the blunt instrument approach and removed > the > > > directory (which is the imported kudu code) from tidying consideration > > the > > > same way we do gutil in run_clang_tidy.sh. Nothing else seemed to work > as > > > one might think it should. > > > > > > On 12 July 2017 at 17:46, Jim Apple <[email protected]> wrote: > > > > > > > The clang-diagnostics are, IIRC, also enabled by the -W flags. You > > could > > > > try turning all warnings off via compiler flags. > > > > > > > > There is also a tool that auto-fixes clang-tidy warnings, but only > some > > > of > > > > them, and I never got even that much to work :-/ > > > > > > > > On Wed, Jul 12, 2017 at 5:24 PM, Henry Robinson <[email protected]> > > > wrote: > > > > > > > > > That does not, for whatever reason, actually disable > > > clang-diagnostic-*. > > > > I > > > > > don't know why either :/ > > > > > > > > > > On 12 July 2017 at 17:15, Jim Apple <[email protected]> wrote: > > > > > > > > > > > What about "diagnostic-henry-thinks-will- > > > > never-fire,-*,-clang-diagnosti > > > > > > c-*"? > > > > > > > > > > > > > > > > > > > > > > > > On Wed, Jul 12, 2017 at 5:01 PM, Henry Robinson < > [email protected]> > > > > > wrote: > > > > > > > > > > > > > Has anyone found a way to disable all clang-tidy checks for a > > > > > directory? > > > > > > > > > > > > > > I've tried a directory-specific .clang-tidy file with > > > > > > > > > > > > > > --- > > > > > > > Checks: "-*" > > > > > > > > > > > > > > but that causes clang-tidy to exit with an error (because I > > didn't > > > > > > > configure any checks). So I tried adding one check that I > thought > > > > would > > > > > > > never fire. But that silently re-enables a bunch of > > > clang-diagnostic* > > > > > > > checks that I don't want. > > > > > > > > > > > > > > This happens when running: > > > > > > > > > > > > > > git diff HEAD~1 | > > > > > > > "${IMPALA_TOOLCHAIN}/llvm-${IMPALA_LLVM_VERSION}/share/clan > > > > > > > g/clang-tidy-diff.py" > > > > > > > -clang-tidy-binary > > > > > > > "${IMPALA_TOOLCHAIN}/llvm-${IMPALA_LLVM_VERSION}/bin/ > clang-tidy" > > > -p > > > > 1 > > > > > > > > > > > > > > per > > > > > > > https://cwiki.apache.org/confluence/pages/viewpage.action? > > > > > > pageId=65868536 > > > > > > > > > > > > > > Any ideas? Am I running clang-tidy wrong? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > Henry Robinson > > > Software Engineer > > > Cloudera > > > 415-994-6679 > > > > > > > > > > > -- > > Todd Lipcon > > Software Engineer, Cloudera > > > -- Todd Lipcon Software Engineer, Cloudera
