Switch to C++17 as minimum requirement

Author: joto

.github/workflows/ci.yml

@@ -22,23 +22,17 @@ jobs:
           - "fedora:43"
         build_type: [Dev]
         cpp_compiler: [g++]
-        cpp_version: [c++14]
+        cpp_version: [c++17]
         include:
           - image: "ubuntu:22.04"
             CXXFLAGS: -Wno-stringop-overread -Wno-array-bounds
           - image: "ubuntu:24.04"
             CXXFLAGS: -Wno-stringop-overread -Wno-array-bounds
-          - image: "debian:bookworm"
-            cpp_version: c++17
           - image: "debian:bookworm"
             cpp_version: c++20
           - image: "debian:bookworm"
             c_compiler: clang
             cpp_compiler: clang++
-          - image: "debian:bookworm"
-            c_compiler: clang
-            cpp_compiler: clang++
-            cpp_version: c++17
           - image: "debian:bookworm"
             c_compiler: clang
             cpp_compiler: clang++
@@ -47,19 +41,12 @@ jobs:
             build_type: RelWithDebInfo
           - image: "debian:trixie"
             CXXFLAGS: -Wno-stringop-overread -Wno-array-bounds
-          - image: "debian:trixie"
-            cpp_version: c++17
-            CXXFLAGS: -Wno-stringop-overread -Wno-array-bounds
           - image: "debian:trixie"
             cpp_version: c++20
             CXXFLAGS: -Wno-stringop-overread -Wno-array-bounds
           - image: "debian:trixie"
             c_compiler: clang
             cpp_compiler: clang++
-          - image: "debian:trixie"
-            c_compiler: clang
-            cpp_compiler: clang++
-            cpp_version: c++17
           - image: "debian:trixie"
             c_compiler: clang
             cpp_compiler: clang++
@@ -121,10 +108,6 @@ jobs:
             nlohmann-json3-dev \
             pandoc \
             zlib1g-dev
-      - name: Install compiler
-        shell: bash
-        if: matrix.cpp_compiler == 'clang++-14'
-        run: apt-get install -yq --no-install-suggests --no-install-recommends clang-14
       - name: Prepare container (dnf)
         shell: bash
         if: startsWith(matrix.image, 'fedora:')

.github/workflows/clang-tidy.yml

@@ -22,7 +22,7 @@ jobs:
         BUILD_TYPE: Dev
         CC: clang-${{ matrix.clang }}
         CXX: clang++-${{ matrix.clang }}
-        CPP_VERSION: c++14
+        CPP_VERSION: c++17
         APT_LISTCHANGES_FRONTEND: none
         DEBIAN_FRONTEND: noninteractive
     steps:

CMakeLists.txt

@@ -87,7 +87,7 @@ if(CPPCHECK)
     # cpp doesn't find system includes for some reason, suppress that report
     set(CPPCHECK_OPTIONS ${CPPCHECK_OPTIONS} --suppress=missingIncludeSystem)
 
-    add_custom_target(cppcheck ${CPPCHECK} --std=c++14 ${CPPCHECK_OPTIONS} ${CMAKE_SOURCE_DIR}/src/*pp)
+    add_custom_target(cppcheck ${CPPCHECK} --std=c++17 ${CPPCHECK_OPTIONS} ${CMAKE_SOURCE_DIR}/src/*pp)
 else()
     message(STATUS "Looking for cppcheck - not found")
     message(STATUS "  Build target 'cppcheck' will not be available")
@@ -113,12 +113,12 @@ endif()
 
 #-----------------------------------------------------------------------------
 #
-#  Decide which C++ version to use (Minimum/default: C++14).
+#  Decide which C++ version to use (Minimum/default: C++17).
 #
 #-----------------------------------------------------------------------------
 if(NOT MSVC)
     if(NOT USE_CPP_VERSION)
-        set(USE_CPP_VERSION c++14)
+        set(USE_CPP_VERSION c++17)
     endif()
     message(STATUS "Use C++ version: ${USE_CPP_VERSION}")
     add_compile_options(-std=${USE_CPP_VERSION})

README.md

@@ -11,7 +11,7 @@ Official web site: https://osmcode.org/osmium-tool/
 
 ## Prerequisites
 
-You need a C++14 compliant compiler.
+You need a C++17 compliant compiler.
 
 You also need the following libraries: