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