[cmake-developers] Generator expressions for install destination

[Robert Goulet]

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.king at kitware.com] 
Sent: Tuesday, September 22, 2015 1:04 PM
To: Robert Goulet <Robert.Goulet at autodesk.com>
Cc: cmake-developers at cmake.org; Stephen Kelly <steveire at 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