Hi, Here's the patch to add generator expressions to the install command DESTINATION option. Let me know if that's good to go.
>@@ -216,6 +216,7 @@ void
>cmScriptGenerator::GenerateScriptActionsPerConfig(std::ostream& os,
> i != this->ConfigurationTypes->end(); ++i)
> {
> const char* config = i->c_str();
>+ this->ConfigurationName = config;
> if(this->GeneratesForConfig(config))
> {
> // Generate a per-configuration block.
>
>This should not be needed if things are factored correctly.
>Everything in that block already passes "config" through as a parameter.
None of the places where I use GetDestination, except in
cmInstallTargetGenerator, receives a config in parameter. An ideally, the
ConfigurationName member should not even exist, but that will force all places
to pass the config as a parameter. Imho it's better to keep refactoring in a
separate patch.
-Robert Goulet
install-dest-genex.patch
Description: install-dest-genex.patch
-- 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
