D5865: Add missing KDE_ENABLE_NAMED_OPERATORS function

René J.V. Bertin noreply at phabricator.kde.org
Mon May 29 22:10:35 UTC 2017


rjvbb added a comment.


  In https://phabricator.kde.org/D5865#112697, @thiago wrote:
  
  > Why are we spending time in named operators? It's much easier to fix the source code not to use them and then the problem goes away.
  
  
  This is about adding a convenience macro for projects that have dependencies using named operators, like certain Boost modules.

INLINE COMMENTS

> KDECompilerSettings.cmake:216-218
>  if (CMAKE_C_COMPILER_ID STREQUAL "GNU" OR
>          CMAKE_C_COMPILER_ID MATCHES "Clang" OR
>          (CMAKE_C_COMPILER_ID STREQUAL "Intel" AND NOT WIN32))

Shouldn't this be checking for CMAKE_CXX_COMPILER_ID?

REPOSITORY
  R240 Extra CMake Modules

REVISION DETAIL
  https://phabricator.kde.org/D5865

To: rjvbb, #frameworks, #build_system, cgilles, kfunk
Cc: thiago, kfunk
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20170529/ac0f3b68/attachment.html>


More information about the Kde-buildsystem mailing list