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