Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update ESMF (partially) and MAPL from spack/develop #313

Merged

Conversation

climbfuji
Copy link
Collaborator

@climbfuji climbfuji commented Aug 29, 2023

Description

For ESMF, manually copy the new version 8.5.0 to our fork and mark 8.3.0b09 as deprecated. There is no clean way to cherry-pick these changes, because our fork has many updates (improvements) that we need to send back to spack/develop first. (I am volunteering to do that after this PR is merged.)

For MAPL, cherry-pick the latest commit (entire file - git checkout spack/develop -- var/spack/repos/builtin/packages/mapl/package.py) so that the file is identical to what's in spack/develop.

Corresponding spack-stack PR is JCSDA/spack-stack#752

Issue(s) addressed

Working towards JCSDA/spack-stack#753

Dependencies

n/a

Impact

Expected impact on downstream repositories: UFS and JEDI applications using ESMF/MAPL

Checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have run the unit tests before creating the PR

@climbfuji climbfuji changed the title Update ESMF (partially) and MAPL from spack/develop DRAFT - Update ESMF (partially) and MAPL from spack/develop Aug 29, 2023
@climbfuji climbfuji changed the title DRAFT - Update ESMF (partially) and MAPL from spack/develop Update ESMF (partially) and MAPL from spack/develop Aug 29, 2023
@climbfuji climbfuji marked this pull request as ready for review August 29, 2023 15:49
@climbfuji climbfuji self-assigned this Aug 29, 2023
@climbfuji climbfuji added the INFRA JEDI Infrastructure label Aug 29, 2023
@climbfuji
Copy link
Collaborator Author

@mathomp4 Ok to merge this? We can bring in your pflogger update as a follow-up.

@mathomp4
Copy link
Collaborator

@mathomp4 Ok to merge this? We can bring in your pflogger update as a follow-up.

I approved. Hopefully we can figure out the pflogger thing soon...it's very useful for those using MAPL!

@climbfuji climbfuji merged commit 1a4db10 into JCSDA:jcsda_emc_spack_stack Aug 29, 2023
13 checks passed
@climbfuji climbfuji deleted the feature/update_esmf_mapl branch August 29, 2023 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INFRA JEDI Infrastructure
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

4 participants