Add C++11 as build requirement in CMake (#48)

* Add C++11 as build requirement Signed-off-by: Uilian Ries <[email protected]> * add awkward-cpp to deps to debug CI failure on Windows * try version restriction instead * add quotes --------- Signed-off-by: Uilian Ries <[email protected]> Co-authored-by: Sam Gillingham <[email protected]>
ubarsc · Dec 13, 2023 · 5e362dd · 5e362dd
1 parent 659d893
commit 5e362dd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git 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
@@ -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}