----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115378/#review48985 -----------------------------------------------------------
This review has been submitted with commit 8feefd08de064de173fbfcef64a922087f9861fc by Alex Merry to branch master. - Commit Hook On Feb. 4, 2014, 12:26 p.m., Alex Merry wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115378/ > ----------------------------------------------------------- > > (Updated Feb. 4, 2014, 12:26 p.m.) > > > Review request for Build System, Extra Cmake Modules, KDE Frameworks, and > Andrius da Costa Ribas. > > > 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 > ----- > > kde-modules/KDECompilerSettings.cmake > 335e1270d19f8342e41b22e7081dea3f7ac0fbfc > > Diff: https://git.reviewboard.kde.org/r/115378/diff/ > > > Testing > ------- > > > Thanks, > > Alex Merry > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel