minimum CMake version - Githubissues

geoschem / GCST-internal

Dummy repo for GCST project management

0 stars 0 forks source link

minimum CMake version #4

Closed LiamBindle closed 4 years ago

LiamBindle commented 4 years ago

I noticed the HEMCO CMakeLists declare cmake_minimum_required(VERSION 3.5). Later the root CMakeLists.txt uses the OpenMP::OpenMP_Fortran target (imported by find_package(OpenMP REQUIRED). OpenMP::OpenMP_Fortran is only available in CMake >= 3.9. We should either

Use OpenMP_<lang>_FLAGS and OpenMP_<lang>_LIBRARIES instead of OpenMP::OpenMP_Fortran
Change the minimum required version to 3.9

LiamBindle commented 4 years ago

This is fixed on the latest dev/GC_13.0.0 branch of HEMCO. 17742e62b9e16d02e78cb62b945a0d833f25df87