rjvbb added inline comments.
INLINE COMMENTS
> apol wrote in KDECompilerSettings.cmake:195
> Why can't we just use check_cxx_compiler_flag?
We could of course. I worked under the assumption that is not done currently
because checking only ID+version is faster. CMake's check_cxx_compiler_flag is
more expensive because it invokes the compiler.
And it can fail, for instance if the user adds an option for which the compiler
emits diagnostics output. That will be true for the "clang on Mac" situation
too, but why generalise brittleness if you don't have to?
> apol wrote in KDECompilerSettings.cmake:203
> Why have a macro that only works on Apple?
? This is a macro that encapsulates a Mac-specific peculiarity but otherwise
should work everywhere. Did I miss something?!
NB: `${_RESULT}` is not going to be set if `COMPILER_OK` isn't set first,
meaning the macro should always "return false" when a flag isn't supported,
even if check_cxx_compiler_flag isn't used.
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D16894
To: rjvbb, #build_system
Cc: apol, kde-frameworks-devel, kde-buildsystem, #build_system, michaelh,
ngraham, bruns
