Closed DusanJovic-NOAA closed 4 years ago
${NETCDF_INCLUDES} ${MPI_Fortran_INCLUDE_PATH} ${NETCDF_INCLUDES_F90}
are specified twice as include directories:
Here: https://github.com/NOAA-EMC/UFS_UTILS/blob/8934efa6474d95ff0dbf5764f26d3389d85e7bf5/sorc/chgres_cube.fd/CMakeLists.txt#L20 and here https://github.com/NOAA-EMC/UFS_UTILS/blob/8934efa6474d95ff0dbf5764f26d3389d85e7bf5/sorc/chgres_cube.fd/CMakeLists.txt#L33
I think these two target_include_directories can be merged.
target_include_directories
Fixed in https://github.com/NOAA-EMC/UFS_UTILS/commit/ac8bfd7a2492894b06b2222a33728bdc451f08f9
are specified twice as include directories:
Here: https://github.com/NOAA-EMC/UFS_UTILS/blob/8934efa6474d95ff0dbf5764f26d3389d85e7bf5/sorc/chgres_cube.fd/CMakeLists.txt#L20 and here https://github.com/NOAA-EMC/UFS_UTILS/blob/8934efa6474d95ff0dbf5764f26d3389d85e7bf5/sorc/chgres_cube.fd/CMakeLists.txt#L33
I think these two
target_include_directories
can be merged.