From 5e362dd0569ee1addb8fd8375abe19d57b515b51 Mon Sep 17 00:00:00 2001 From: Uilian Ries Date: Thu, 14 Dec 2023 00:00:54 +0100 Subject: [PATCH] Add C++11 as build requirement in CMake (#48) * Add C++11 as build requirement Signed-off-by: Uilian Ries * add awkward-cpp to deps to debug CI failure on Windows * try version restriction instead * add quotes --------- Signed-off-by: Uilian Ries Co-authored-by: Sam Gillingham --- .github/workflows/test-build.yml | 2 +- src/CMakeLists.txt | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/test-build.yml b/.github/workflows/test-build.yml index 48d3d99..06d4179 100644 --- a/.github/workflows/test-build.yml +++ b/.github/workflows/test-build.yml @@ -85,7 +85,7 @@ jobs: - name: Install Dependencies shell: bash -el {0} run: | - conda install cmake awkward hdf5 python numpy libgdal pybind11 + conda install cmake "awkward>=2.5.0" hdf5 python numpy libgdal pybind11 - name: Cmake shell: bash -el {0} run: | diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index c205ba8..e41fea1 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -31,6 +31,7 @@ source_group("include_kea" FILES ${LIBKEA_H}) # Build, link and install library add_library(${LIBKEA_LIB_NAME} ${LIBKEA_CPP} ${LIBKEA_H} ) target_link_libraries(${LIBKEA_LIB_NAME} PRIVATE ${HDF5_LIBRARIES}) +target_compile_features(${LIBKEA_LIB_NAME} PUBLIC cxx_std_11) include(GenerateExportHeader) generate_export_header(${LIBKEA_LIB_NAME}