Re: [CMake] Using clang-tidy with cmake 3.6.0

2016-07-24 Thread Theodoros Theodoridis

Oh, I am sorry. For some reason I did not notice the second reply.

Thanks
--

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake


Re: [CMake] Using clang-tidy with cmake 3.6.0

2016-07-24 Thread Theodoros Theodoridis

Thanks for the answer. However, I still can't get it to work:

I tried using set_target_properties(target PROPERTIES CXX_CLANG_TIDY 
"/usr/local/bin/clang-tidy -checks='llvm*'")

and the error I got was:


/usr/bin/cmake -E __run_iwyu --tidy="/usr/local/bin/clang-tidy 
-checks='llvm*'" --source=

...
Error running '/usr/local/bin/clang-tidy -checks='llvm*''


On 07/20/2016 10:02 PM, Robert Maynard wrote:

I believe the correct form is:

set_target_properties(example PROPERTIES CXX_CLANG_TIDY
"${path_to_tidy} -some -args")

On Wed, Jul 20, 2016 at 9:02 AM, Theodoros Theodoridis
 wrote:

Hello everyone,

I am trying to use the new _CLANG_TIDY property but I can't figure
out the proper syntax. How can I specify the checks I want?

I tried setting the checks in set_target_properties(target PROPERTIES
CXX_CLANG_TIDY "checks to be performed") but cmake would complain with
"wrong number of arguments".

I would appreciate any help.

Thanks,
Theodor

--

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at:
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake


--

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake


Re: [CMake] Using clang-tidy with cmake 3.6.0

2016-07-20 Thread Bill Hoffman

On 7/20/2016 4:02 PM, Robert Maynard wrote:

I believe the correct form is:

set_target_properties(example PROPERTIES CXX_CLANG_TIDY
"${path_to_tidy} -some -args")

No, that won't work.

You want it to be a semi-colon separated list of arguments.  Not space 
separated.


https://cmake.org/cmake/help/v3.6/prop_tgt/LANG_CLANG_TIDY.html#prop_tgt:_CLANG_TIDY


In a ctest script it would be done something like this:

set(DASHBOARD_CLANG_TIDAY
"/path/to/clang-tidy" "-arg1" "-arg2")


Then put this in the cache:
"CMAKE_CXX_CLANG_TIDY:STRING=${DASHBOARD_CLANG_TIDY}"

-Bill
--

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake


Re: [CMake] Using clang-tidy with cmake 3.6.0

2016-07-20 Thread Robert Maynard
I believe the correct form is:

set_target_properties(example PROPERTIES CXX_CLANG_TIDY
"${path_to_tidy} -some -args")

On Wed, Jul 20, 2016 at 9:02 AM, Theodoros Theodoridis
 wrote:
> Hello everyone,
>
> I am trying to use the new _CLANG_TIDY property but I can't figure
> out the proper syntax. How can I specify the checks I want?
>
> I tried setting the checks in set_target_properties(target PROPERTIES
> CXX_CLANG_TIDY "checks to be performed") but cmake would complain with
> "wrong number of arguments".
>
> I would appreciate any help.
>
> Thanks,
> Theodor
>
> --
>
> Powered by www.kitware.com
>
> Please keep messages on-topic and check the CMake FAQ at:
> http://www.cmake.org/Wiki/CMake_FAQ
>
> Kitware offers various services to support the CMake community. For more
> information on each offering, please visit:
>
> CMake Support: http://cmake.org/cmake/help/support.html
> CMake Consulting: http://cmake.org/cmake/help/consulting.html
> CMake Training Courses: http://cmake.org/cmake/help/training.html
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Follow this link to subscribe/unsubscribe:
> http://public.kitware.com/mailman/listinfo/cmake
-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake


[CMake] Using clang-tidy with cmake 3.6.0

2016-07-20 Thread Theodoros Theodoridis

Hello everyone,

I am trying to use the new _CLANG_TIDY property but I can't figure
out the proper syntax. How can I specify the checks I want?

I tried setting the checks in set_target_properties(target PROPERTIES
CXX_CLANG_TIDY "checks to be performed") but cmake would complain with
"wrong number of arguments".

I would appreciate any help.

Thanks,
Theodor

--

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake