Please try new patch in attachment, this might fix it. I'm now using both the config name and the implib boolean for the map key. I didn't use std::tie to implement operator< since I remember you told me to not use C++11 features in CMake code.
Let me know if that works out. Thanks. -----Original Message----- From: Brad King [mailto:brad.k...@kitware.com] Sent: Tuesday, August 18, 2015 10:44 AM To: Robert Goulet <robert.gou...@autodesk.com> Cc: cmake-developers@cmake.org Subject: Re: Recursion in OUTPUT_NAME genex On 08/17/2015 11:59 AM, Robert Goulet wrote: > I'm assuming that master branch is 3.4? Yes. It's not yet in 'master' though. > cmGeneratorTarget: Avoid recursion in GetOutputName method > http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=e66a7d69 I had to revert it due to nightly testing failures. The problem is that the output name map you created needs to have two values for each entry. The result is different for implib and !implib. Please revise. Thanks, -Brad
output-name-genex-recursion-fix.patch
Description: output-name-genex-recursion-fix.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
