[cmake-developers] Recursion in OUTPUT_NAME genex

[Robert Goulet]

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.king at kitware.com] 
Sent: Tuesday, August 18, 2015 10:44 AM
To: Robert Goulet <Robert.Goulet at autodesk.com>
Cc: cmake-developers at 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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: output-name-genex-recursion-fix.patch
Type: application/octet-stream
Size: 4571 bytes
Desc: output-name-genex-recursion-fix.patch
URL: <http://public.kitware.com/pipermail/cmake-developers/attachments/20150818/0912299d/attachment.obj>