<table><tr><td style="">rjvbb created this revision.<br />rjvbb added projects: Build System, Frameworks.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D5089" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>KDECompilerSettings.cmake and KDEFrameworksCompilerSettings.cmake fail to consider the fact that cmake considers Apple's clang compiler to be different (differences that exist but are largely irrelevant for KDE and the ECM in particular).</p>

<p>I just ran into a failure building the KJs 5.32.0 framework as a result, using the system/Xcode clang compiler for OS X 10.9.5 . The build system had failed to put the compiler into C++11 mode.</p>

<p>This patch address the issue by checking the CMAKE_*_COMPILER_ID against both "Clang" and "AppleClang". A more elegant approach may be possible with regexp pattern matching but might cause problems if other clang flavours exist and match that shouldn't .</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>With this patch KJs builds without further errors. I will update this ticket should I run into unforeseen side-effects.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R240 Extra CMake Modules</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5089" rel="noreferrer">https://phabricator.kde.org/D5089</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kde-modules/KDECompilerSettings.cmake<br />
kde-modules/KDEFrameworkCompilerSettings.cmake</div></div></div><br /><div><strong>To: </strong>rjvbb, Build System, Frameworks<br /><strong>Cc: </strong>kde-mac, Frameworks, Build System<br /></div>