Review Request 115378: Resolve some FIXMEs related to MSVC and Intel

Alex Merry kde at randomguy3.me.uk
Wed Jan 29 16:34:00 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115378/
-----------------------------------------------------------

(Updated Jan. 29, 2014, 4:34 p.m.)


Review request for Build System, Extra Cmake Modules, KDE Frameworks, and Andrius da Costa Ribas.


Changes
-------

Rebase against latest master


Repository: extra-cmake-modules


Description
-------

Use the MSVC variable directly, instead of the compiler ID

This is cleaner and easier to read.

Resolve some FIXMEs related to MSVC and Intel

In particular, MSVC (and Intel on Windows) have no equivalent of the
-std flag to set the language standard, and Intel does not appear to
produce the warnings that were disabled for MSVC.


Diffs (updated)
-----

  kde-modules/KDECompilerSettings.cmake 92d1f823f0b257a828b3acc98e77f386dd587ff2 

Diff: https://git.reviewboard.kde.org/r/115378/diff/


Testing
-------


Thanks,

Alex Merry

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20140129/557ea055/attachment.html>


More information about the Kde-buildsystem mailing list