Skip to content

Manual update of ESMF from spack develop 2025/05/07#538

Merged
climbfuji merged 2 commits intoJCSDA:spack-stack-devfrom
climbfuji:feature/manual_update_esmf_from_spack_dev_20250507
May 7, 2025
Merged

Manual update of ESMF from spack develop 2025/05/07#538
climbfuji merged 2 commits intoJCSDA:spack-stack-devfrom
climbfuji:feature/manual_update_esmf_from_spack_dev_20250507

Conversation

@climbfuji
Copy link
Copy Markdown
Collaborator

@climbfuji climbfuji commented May 7, 2025

Description

To address build errors for the latest ESMF beta snapshot (see, for example, https://github.com/JCSDA/spack-stack/actions/runs/14887254221/job/41810037631?pr=1632), we need to pull in updates from spack develop. Unfortunately, there's been a fair bit of reorganization and PRs that touch many more packages than just ESMF that makes it impossible to do this by cherry-picking.

I therefore hand-picked all applicable differences for the ESMF package. With the exception of the beta snapshots and the new Fortran compiler identification code (requires spack code we do not have yet), the esmf/package.py here is identical to the version in spack develop as of 2025/05/07.

This is needed for JCSDA/spack-stack#1632 as a follow-up to #537.

@climbfuji climbfuji force-pushed the feature/manual_update_esmf_from_spack_dev_20250507 branch from f84f346 to 2c87f43 Compare May 7, 2025 17:09
@climbfuji climbfuji marked this pull request as ready for review May 7, 2025 18:18
@climbfuji climbfuji self-assigned this May 7, 2025
Copy link
Copy Markdown
Collaborator

@theurich theurich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice.

@climbfuji climbfuji merged commit c44c79e into JCSDA:spack-stack-dev May 7, 2025
16 checks passed
@climbfuji climbfuji deleted the feature/manual_update_esmf_from_spack_dev_20250507 branch May 7, 2025 19:47
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.

3 participants