[CMake] CMAKE_<LANG>_FLAGS added to link rule

[Verweij, Arjen]

Brad,

That bit of code is #IFDEF'ed for __APPLE__. I doubt it is causing the problem on AIX.

Regards,
Arjen

>-----Original Message-----
>From: cmake-bounces at cmake.org [mailto:cmake-bounces at cmake.org] On Behalf
>Of Verweij, Arjen
>Sent: maandag 12 juli 2010 20:47
>To: Brad King
>Cc: cmake at cmake.org; Bill Hoffman
>Subject: Re: [CMake] CMAKE_<LANG>_FLAGS added to link rule
>
>Brad,
>
>That sounds fine, I'm already in the habit of specifying link flags per
>language anyway. In fact I will take a looksie and recompile cmake with
>your suggested fix.
>
>Regards,
>Arjen
>
>>Looking at the implementation I see it gets architecture flags on
>>the Mac (-arch and -isysroot), which is consistent with the -m64
>>type flags I mentioned before.  However it looks to me like things
>>have been refactored enough to simply move these into the link-only
>>flags with a one-line change.  The change would not solve the original
>>problem but would enable your workaround.
>>
>>Bill?
>>
>>-Brad
>>
>>
>>diff --git a/Source/cmMakefileExecutableTargetGenerator.cxx
>>b/Source/cmMakefileExecutableTargetGenerator.cxx
>>index 93c981a..cb41e74 100644
>>--- a/Source/cmMakefileExecutableTargetGenerator.cxx
>>+++ b/Source/cmMakefileExecutableTargetGenerator.cxx
>>@@ -229,7 +229,7 @@ void
>>cmMakefileExecutableTargetGenerator::WriteExecutableRule(bool relink)
>>   // Add language feature flags.
>>   this->AddFeatureFlags(flags, linkLanguage);
>>
>>-  this->LocalGenerator->AddArchitectureFlags(flags, this->Target,
>>+  this->LocalGenerator->AddArchitectureFlags(linkFlags, this->Target,
>>                                              linkLanguage, this-
>>>ConfigName);
>>
>>   // Add target-specific linker flags.
>_______________________________________________
>Powered by www.kitware.com
>
>Visit other Kitware open-source projects at
>http://www.kitware.com/opensource/opensource.html
>
>Please keep messages on-topic and check the CMake FAQ at:
>http://www.cmake.org/Wiki/CMake_FAQ
>
>Follow this link to subscribe/unsubscribe:
>http://www.cmake.org/mailman/listinfo/cmake