On 07/16/2014 11:55 AM, Daniele E. Domenichelli wrote:
> Please review the topic CMakePackageConfigHelpers_build_tree.
> 
> This patch adds an option BUILD_TREE to configure_package_config_file.

The changes look good to me, though I'd appreciate feedback from
others who have worked on this module before.

In the documentation formatting commit, I see blocks like:

> +# .. command:: configure_package_config_file
> +#
> +# Create a config file for a project::

The definition of the command should be in an indented block so
that reST knows it is part of the explicit markup block.

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