Skip to content

Remove local copy of FindESMF.cmake and rename esmf target to ESMF::ESMF#1482

Merged
JessicaMeixner-NOAA merged 7 commits into
NOAA-EMC:dev/ufs-weather-modelfrom
DusanJovic-NOAA:remove_findesmf
Nov 25, 2025
Merged

Remove local copy of FindESMF.cmake and rename esmf target to ESMF::ESMF#1482
JessicaMeixner-NOAA merged 7 commits into
NOAA-EMC:dev/ufs-weather-modelfrom
DusanJovic-NOAA:remove_findesmf

Conversation

@DusanJovic-NOAA
Copy link
Copy Markdown
Contributor

Pull Request Summary

Remove local copy of FindESMF.cmake and rename esmf target to ESMF::ESMF

Description

ufs-weather-model now uses the FindESMF.cmake module provided by the ESMF library, so there's no need to keep a local copy. Also, the name of the imported esmf teget is changed to ESMF::ESMF.

Issue(s) addressed

Commit Message

Remove local copy of FindESMF.cmake and rename esmf target to ESMF::ESMF

Check list

Testing

  • How were these changes tested? ufs-weather-model regression test
  • Are the changes covered by regression tests? (If not, why? Do new tests need to be added?)
  • Have the matrix regression tests been run (if yes, please note HPC and compiler)?
  • Please indicate the expected changes in the regression test output, (Note the list of known non-identical tests.)
  • Please provide the summary output of matrix.comp (matrix.Diff.txt, matrixCompFull.txt and matrixCompSummary.txt):

@JessicaMeixner-NOAA
Copy link
Copy Markdown
Collaborator

Noting on this PR that it should be merged w/the UFS level -- and will be coordinated w/other component changes, etc.

@JessicaMeixner-NOAA JessicaMeixner-NOAA merged commit af7ecf2 into NOAA-EMC:dev/ufs-weather-model Nov 25, 2025
@DusanJovic-NOAA DusanJovic-NOAA deleted the remove_findesmf branch December 1, 2025 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants