D5865: Add missing KDE_ENABLE_NAMED_OPERATORS function

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


rjvbb added a comment.


  In https://phabricator.kde.org/D5865#112743, @thiago wrote:
  
  > Fix the source code.
  >
  > If certain third-party libraries can't compile under MSVC 2015 (or even 2017!) with default compiler options, they don't deserve to be used. Blacklist them.
  
  
  That may work for Qt but is not an acceptable attitude for ECM, IMHO - and MSVC is problematic enough to build KF5 code for other reasons not to jump through hoops for it. You cannot expect FOSS projects to avoid Boost because it uses less common feature of the standard, big ole bad M$ cannot be bothered to write a proper compiler. I'd be all for blacklisting such compilers, if anything had to be blacklisted.
  The projects I'm aware of that use "incriminated" boost code do provide MSWin builds but using a properly standards-compliant compiler.
  
  My proposal is thus to drop MSVC support in the new macro, printing a warning should be enough.

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/224ed78e/attachment.html>


More information about the Kde-buildsystem mailing list