On 02/19/2016 08:09 PM, Philippe Proulx wrote:
> Signed-off-by: Philippe Proulx <[email protected]>
> ---
>  Modules/FindLTTngUST.cmake | 88 
> ++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 88 insertions(+)
>  create mode 100644 Modules/FindLTTngUST.cmake

Thanks for working on this.

Please look at adding imported targets as described here:

  https://cmake.org/cmake/help/v3.5/manual/cmake-developer.7.html#find-modules

See the FindPNG module for an example.  Also please add a test
case.  Grep in the Tests directory for "FindPNG" for an example.

Also please udpate Tests/CMakeOnly/AllFindModules/CMakeLists.txt
at the bottom to check that the version string is available for
this package.

Thanks,
-Brad

-- 

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-developers

Reply via email to