D24841: Use modern way to set the C/CXX standard

2019-10-23 Thread Friedrich W. H. Kossebau
kossebau added a comment. In D24841#552842 , @vonreth wrote: > Where are those tests running? I'm only aware of https://build.kde.org/job/Frameworks/job/extra-cmake-modules/ The tests are currntly sadly skipped on CI, compare T11858

D24841: Use modern way to set the C/CXX standard

2019-10-23 Thread Hannah von Reth
vonreth added a comment. In D24841#552600 , @cgiboudeaux wrote: > Another issue caused by the new CMake 3.5 dependency, some tests fail: > > > > > > > 59 - ecm_setup_version-old_simple (Failed) > > 60 -

D24841: Use modern way to set the C/CXX standard

2019-10-23 Thread Christophe Giboudeaux
cgiboudeaux added a comment. Another issue caused by the new CMake 3.5 dependency, some tests fail: > > > 59 - ecm_setup_version-old_simple (Failed) > 60 - ecm_setup_version-old_soversion (Failed) > 61 - ecm_setup_version-old_version_file (Failed) > 62 -

D24841: Use modern way to set the C/CXX standard

2019-10-23 Thread David Faure
dfaure added a comment. Does https://phabricator.kde.org/D24882 help? (Not tested) REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D24841 To: vonreth, dfaure, cullmann Cc: cgiboudeaux, aacid, chehrlic, kde-frameworks-devel, kde-buildsystem, LeGast00n,

D24841: Use modern way to set the C/CXX standard

2019-10-23 Thread David Faure
dfaure added a comment. Where did -std=gnu++14 come from? The old code above certainly didn't set it. Maybe some projects were doing `set(CMAKE_CXX_STANDARD 14)` before including KDECompilerSettings? We could test the var here to avoid overwriting it... REPOSITORY R240 Extra CMake

D24841: Use modern way to set the C/CXX standard

2019-10-23 Thread Christophe Giboudeaux
cgiboudeaux added a comment. I'm seeing build failures in several repositories seemingly caused by 6e3c794 (eg akonadi, kasync) Building with clang9, before: `-std=c++0x -std=gnu++14` are set After:

D24841: Use modern way to set the C/CXX standard

2019-10-22 Thread Hannah von Reth
This revision was automatically updated to reflect the committed changes. Closed by commit R240:6e3c794e7c63: Use modern way to set the C/CXX standad (authored by vonreth). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D24841?vs=68488=68560#toc REPOSITORY R240 Extra CMake Modules

D24841: Use modern way to set the C/CXX standard

2019-10-22 Thread Christophe Giboudeaux
cgiboudeaux retitled this revision from "Use modern way to set the C/CXX standad" to "Use modern way to set the C/CXX standard". cgiboudeaux edited the summary of this revision. REPOSITORY R240 Extra CMake Modules BRANCH cmake_c_standard REVISION DETAIL https://phabricator.kde.org/D24841