Thanks, once this is accepted in master I will send you my updated 
install(FILES) with genex support. I removed the GetDestination() signature 
since I agree it's not needed and might be confusing.

-----Original Message-----
From: Brad King [mailto:brad.k...@kitware.com] 
Sent: Tuesday, September 22, 2015 1:04 PM
To: Robert Goulet <robert.gou...@autodesk.com>
Cc: cmake-developers@cmake.org; Stephen Kelly <steve...@gmail.com>
Subject: Re: Generator expressions for install destination

On 09/22/2015 09:58 AM, Robert Goulet wrote:
> Patch attached for adding makefile to install generators.
> This refactoring is required for install(FILES) genex support, and 
> most likely other install() signatures in the future.

Thanks.  I applied that and merged to 'next' for testing:

 cmInstallGenerator: Add Makefile member
 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=d44cb327

I also extended the topic with some other refactoring it enables:

 cmInstallFilesGenerator: Drop LocalGenerator member  
http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=e4b1728c

 cmInstallTargetGenerator: Simplify using Makefile member  
http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=3f6b267d

Steve, please check whether this runs afoul of your cmState refactoring design.

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