On 07/18/2015 02:48 PM, Clifford Yapp wrote:
> A quick experiment suggests that this will work for associating and
> exposing a source directory with each target - not sure if this dots
> all the i's an crosses all the t's though:
> 
> +  MAKE_STATIC_PROP(SOURCE_DIR);

That looks good to me.  Please extend the patch to also do BINARY_DIR
for completeness and also to update the documentation to cover the new
properties.  The Tests/RunCMake/get_property test should be updated
to cover both too.

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