Re: [CMake] CXX_CLANG_TIDY doesn't handle multiple arguments correctly

2016-11-16 Thread Nikita
Thanks for the info. I've filed a bug about this issue - https://gitlab.kitware.com/cmake/cmake/issues/16435 Regards, Nikita On Thu, Nov 17, 2016 at 1:37 AM, Michael Ellery wrote: > >> On Nov 16, 2016, at 2:32 PM, Nikita wrote: >> >> Indeed, specifying '-line-filter=[]' doesn't cause any troubl

Re: [CMake] CXX_CLANG_TIDY doesn't handle multiple arguments correctly

2016-11-16 Thread Michael Ellery
> On Nov 16, 2016, at 2:32 PM, Nikita wrote: > > Indeed, specifying '-line-filter=[]' doesn't cause any troubles. > Should I file a bug about this? > > Regards, > Nikita > FWIW, I brought up the same issue a few weeks ago: http://stackoverflow.com/questions/40433573/how-can-i-specify-additio

Re: [CMake] CXX_CLANG_TIDY doesn't handle multiple arguments correctly

2016-11-16 Thread Nikita
Indeed, specifying '-line-filter=[]' doesn't cause any troubles. Should I file a bug about this? Regards, Nikita On Thu, Nov 17, 2016 at 1:18 AM, Bill Hoffman wrote: > On 11/16/2016 5:01 PM, Nikita wrote: >> >> It works for me with '-fix' as well. However, it doesn't work with >> "-warnings-as-e

Re: [CMake] CXX_CLANG_TIDY doesn't handle multiple arguments correctly

2016-11-16 Thread Bill Hoffman
On 11/16/2016 5:01 PM, Nikita wrote: It works for me with '-fix' as well. However, it doesn't work with "-warnings-as-errors", '-header-filter', and '-line-filter'. The problem is definitely not in semicolons. It is the * that causes trouble. -Bill -- Powered by www.kitware.com Please keep m

Re: [CMake] CXX_CLANG_TIDY doesn't handle multiple arguments correctly

2016-11-16 Thread Nikita
It works for me with '-fix' as well. However, it doesn't work with "-warnings-as-errors", '-header-filter', and '-line-filter'. The problem is definitely not in semicolons. Regards, Nikita On Thu, Nov 17, 2016 at 12:35 AM, Tiago Macarios wrote: > This works for me: > > CXX_CLANG_TIDY >

Re: [CMake] CXX_CLANG_TIDY doesn't handle multiple arguments correctly

2016-11-16 Thread Tiago Macarios
This works for me: CXX_CLANG_TIDY "/usr/bin/clang-tidy" "-checks=modernize-*,readability-*,performance-*" "-fix" I think the difference are the semicolons in the checks instead of commas. On Wed, Nov 16, 2016 at 1:32 PM, Nikita wrote: > Hi, > > I've noticed that CXX

[CMake] CXX_CLANG_TIDY doesn't handle multiple arguments correctly

2016-11-16 Thread Nikita
Hi, I've noticed that CXX_CLANG_TIDY property doesn't work correctly if I specify several arguments to clang-tidy. Here's an example: set_property( TARGET ${PROJECT_NAME} PROPERTY CXX_CLANG_TIDY clang-tidy;-checks=*;-warnings-as-errors=*) Building a target with this property produces no o