Commit 4b254ed2 authored by Nicolas Fella's avatar Nicolas Fella
Browse files

Require C++17

As discussed in https://mail.kde.org/pipermail/plasma-devel/2020-August/118263.html we agreed that requiring C++17 in Plasma is fine.
There's no code that actually uses C++17 features yet, but specifying the requirement anyway has advantages.

It documents the policy in a much more discoverable place than a wiki page or similar could do
Building in 17 mode allows the compiler to do some stuff implicitly, like guranteed copy elision
parent f45219c9
......@@ -9,6 +9,9 @@ set(QT_MIN_VERSION "5.12.0")
set(KF5_MIN_VERSION "5.73.0")
set(ACCOUNTSQT_DEP_VERSION "1.13")
set(CMAKE_CXX_STANDARD 17)
set(CMAKE_CXX_STANDARD_REQUIRED ON)
find_package(ECM ${KF5_MIN_VERSION} REQUIRED NO_MODULE)
set(CMAKE_MODULE_PATH ${ECM_MODULE_PATH} ${CMAKE_CURRENT_SOURCE_DIR}/cmake/modules)
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment