Skip to content

Commit

Permalink
Merge pull request #750 from GEOS-ESM/hotfix/mathomp4/fix-spack-build
Browse files Browse the repository at this point in the history
Update to Baselibs 7.24.0 and MAPL 2.46.1 for Spack Build support
  • Loading branch information
sdrabenh authored May 15, 2024
2 parents c10f0da + c59e364 commit 30f4a9c
Show file tree
Hide file tree
Showing 5 changed files with 21 additions and 34 deletions.
6 changes: 3 additions & 3 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ parameters:

# Anchors to prevent forgetting to update a version
os_version: &os_version ubuntu20
baselibs_version: &baselibs_version v7.23.0
baselibs_version: &baselibs_version v7.24.0
bcs_version: &bcs_version v11.3.0
tag_build_arg_name: &tag_build_arg_name gcmversion

Expand Down Expand Up @@ -103,9 +103,9 @@ workflows:
bcs_version: *bcs_version
container_name: geosgcm
mpi_name: openmpi
mpi_version: 5.0.0
mpi_version: 5.0.2
compiler_name: gcc
compiler_version: 12.1.0
compiler_version: 13.2.0
image_name: geos-env-bcs
tag_build_arg_name: *tag_build_arg_name
resource_class: xlarge
2 changes: 1 addition & 1 deletion .github/workflows/workflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ jobs:
if: "!contains(github.event.pull_request.labels.*.name, '0 diff trivial')"
runs-on: ubuntu-latest
container:
image: gmao/ubuntu20-geos-env:v7.23.0-intelmpi_2021.6.0-intel_2022.1.0
image: gmao/ubuntu20-geos-env:v7.24.0-intelmpi_2021.6.0-intel_2022.1.0
# Per https://github.com/actions/virtual-environments/issues/1445#issuecomment-713861495
# It seems like we might not need secrets on GitHub Actions which is good for forked
# pull requests
Expand Down
35 changes: 11 additions & 24 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -60,31 +60,18 @@ if (NOT Baselibs_FOUND)
add_definitions(-DH5_HAVE_PARALLEL)
endif()

if (NOT TARGET esmf)
if (DEFINED ENV{esmf_ROOT})
message (STATUS "Found esmf_ROOT in environment: $ENV{esmf_ROOT}")
find_path(ESMF_CMAKE_PATH FindESMF.cmake HINTS "$ENV{esmf_ROOT}/cmake")
message (STATUS "Found FindESMF.cmake in: ${ESMF_CMAKE_PATH}")
if (ESMF_CMAKE_PATH)
message (STATUS "Appending to CMAKE_PREFIX_PATH: ${ESMF_CMAKE_PATH}")
list (APPEND CMAKE_MODULE_PATH ${ESMF_CMAKE_PATH})
endif ()
if (NOT TARGET ESMF::ESMF)
find_package(ESMF 8.6.1 MODULE REQUIRED)
target_link_libraries(ESMF::ESMF INTERFACE MPI::MPI_Fortran)

# GEOS uses lowercase target due to historical reasons but
# the latest FindESMF.cmake file from ESMF produces an ESMF::ESMF target.
if (NOT TARGET esmf)
add_library(esmf ALIAS ESMF::ESMF)
endif ()
if (NOT TARGET ESMF)
add_library(ESMF ALIAS ESMF::ESMF)
endif ()

find_package(ESMF MODULE REQUIRED)

# ESMF as used in MAPL requires MPI
# NOTE: This looks odd because some versions of FindESMF.cmake out in the
# world provide an "esmf" target while others provide "ESMF". So we
# need this ugliness to support both.
if (TARGET esmf)
target_link_libraries(esmf INTERFACE MPI::MPI_Fortran)
else()
target_link_libraries(ESMF INTERFACE MPI::MPI_Fortran)
# MAPL and GEOS use lowercase target due to historical reasons but
# the latest FindESMF.cmake file from ESMF produces an ESMF target.
add_library(esmf ALIAS ESMF)
endif()
endif ()

find_package(GFTL_SHARED REQUIRED)
Expand Down
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@
| [CICE](https://github.com/GEOS-ESM/CICE) | [geos/v0.1.3](https://github.com/GEOS-ESM/CICE/releases/tag/geos%2Fv0.1.3) |
| [CPLFCST_Etc](https://github.com/GEOS-ESM/CPLFCST_Etc) | [v1.0.1](https://github.com/GEOS-ESM/CPLFCST_Etc/releases/tag/v1.0.1) |
| [ecbuild](https://github.com/GEOS-ESM/ecbuild) | [geos/v1.3.0](https://github.com/GEOS-ESM/ecbuild/releases/tag/geos%2Fv1.3.0) |
| [ESMA_cmake](https://github.com/GEOS-ESM/ESMA_cmake) | [v3.45.0](https://github.com/GEOS-ESM/ESMA_cmake/releases/tag/v3.45.0) |
| [ESMA_env](https://github.com/GEOS-ESM/ESMA_env) | [v4.28.1](https://github.com/GEOS-ESM/ESMA_env/releases/tag/v4.28.1) |
| [ESMA_cmake](https://github.com/GEOS-ESM/ESMA_cmake) | [v3.45.1](https://github.com/GEOS-ESM/ESMA_cmake/releases/tag/v3.45.1) |
| [ESMA_env](https://github.com/GEOS-ESM/ESMA_env) | [v4.29.0](https://github.com/GEOS-ESM/ESMA_env/releases/tag/v4.29.0) |
| [FMS](https://github.com/GEOS-ESM/FMS) | [geos/2019.01.02+noaff.8](https://github.com/GEOS-ESM/FMS/releases/tag/geos%2F2019.01.02%2Bnoaff.8) |
| [FVdycoreCubed_GridComp](https://github.com/GEOS-ESM/FVdycoreCubed_GridComp) | [v2.11.1](https://github.com/GEOS-ESM/FVdycoreCubed_GridComp/releases/tag/v2.11.1) |
| [geos-chem](https://github.com/GEOS-ESM/geos-chem) | [geos/v13.0.0-rc1](https://github.com/GEOS-ESM/geos-chem/releases/tag/geos%2Fv13.0.0-rc1) |
Expand All @@ -32,7 +32,7 @@
| [GOCART](https://github.com/GEOS-ESM/GOCART) | [sdr_v2.2.1.1](https://github.com/GEOS-ESM/GOCART/releases/tag/sdr_v2.2.1.1) |
| [HEMCO](https://github.com/GEOS-ESM/HEMCO) | [geos/v2.2.3](https://github.com/GEOS-ESM/HEMCO/releases/tag/geos%2Fv2.2.3) |
| [Icepack](https://github.com/GEOS-ESM/Icepack) | [geos/v0.2.0](https://github.com/GEOS-ESM/Icepack/releases/tag/geos%2Fv0.2.0) |
| [MAPL](https://github.com/GEOS-ESM/MAPL) | [v2.45.0](https://github.com/GEOS-ESM/MAPL/releases/tag/v2.45.0) |
| [MAPL](https://github.com/GEOS-ESM/MAPL) | [v2.46.1](https://github.com/GEOS-ESM/MAPL/releases/tag/v2.46.1) |
| [MITgcm](https://github.com/GEOS-ESM/MITgcm) | [checkpoint68o](https://github.com/GEOS-ESM/MITgcm/releases/tag/checkpoint68o) |
| [MOM5](https://github.com/GEOS-ESM/MOM5) | [geos/5.1.0+1.2.0](https://github.com/GEOS-ESM/MOM5/releases/tag/geos%2F5.1.0%2B1.2.0) |
| [MOM6](https://github.com/GEOS-ESM/MOM6) | [geos/v3.1](https://github.com/GEOS-ESM/MOM6/tree/geos/v3.1) |
Expand Down
6 changes: 3 additions & 3 deletions components.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,13 @@ GEOSgcm:
env:
local: ./@env
remote: ../ESMA_env.git
tag: v4.28.1
tag: v4.29.0
develop: main

cmake:
local: ./@cmake
remote: ../ESMA_cmake.git
tag: v3.45.0
tag: v3.45.1
develop: develop

ecbuild:
Expand Down Expand Up @@ -42,7 +42,7 @@ GEOS_Util:
MAPL:
local: ./src/Shared/@MAPL
remote: ../MAPL.git
tag: v2.45.0
tag: v2.46.1
develop: develop

FMS:
Expand Down

0 comments on commit 30f4a9c

Please sign in to comment.