Skip to content

gfe: Update GFE packages#1782

Merged
tldahlgren merged 9 commits intospack:developfrom
mathomp4:gfe-update-2025Sep30
Oct 16, 2025
Merged

gfe: Update GFE packages#1782
tldahlgren merged 9 commits intospack:developfrom
mathomp4:gfe-update-2025Sep30

Conversation

@mathomp4
Copy link
Copy Markdown
Contributor

@mathomp4 mathomp4 commented Sep 30, 2025

This PR updates these GFE packages:

These are done en masse for two reasons:

  1. These packages now all require CMake 3.24 or higher
  2. The updates to pFUnit, yaFyaml, and pFlogger also have fixes for working with ifx 2025.2

Because of the latter, I've also included all the changes from #1592 by @climbfuji into this PR (including the MAPL 2.62.0 update as it is also ifx 2025.2 related). I did this mainly because I'm not sure which PR will go in first.

Closes #1592
Closes #1454
Closes #1976

@mathomp4
Copy link
Copy Markdown
Contributor Author

mathomp4 commented Oct 1, 2025

I pushed an update as I did a test build with ifx-in-spack and things failed. Luckily, @climbfuji figured out the special sauce in #1592 on how to get spack to do the right thing. I now have pure Fortran code depending on C only because we need GNU cpp !

# Conflicts:
#	repos/spack_repo/builtin/packages/mapl/package.py
@mathomp4
Copy link
Copy Markdown
Contributor Author

mathomp4 commented Oct 7, 2025

I've pulled in the changes from #1454 by @AlexanderRichert-NOAA and I've added the commit hash for the latest ESMA_cmake.

Copy link
Copy Markdown
Contributor

@AlexanderRichert-NOAA AlexanderRichert-NOAA left a comment

Choose a reason for hiding this comment

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

Thanks for adding those resource commit hashes

# Conflicts:
#	repos/spack_repo/builtin/packages/mapl/package.py
#	repos/spack_repo/builtin/packages/pflogger/package.py
#	repos/spack_repo/builtin/packages/yafyaml/package.py
@mathomp4
Copy link
Copy Markdown
Contributor Author

mathomp4 commented Oct 9, 2025

Now that @tldahlgren has merged #1592, I've merged develop into this.

@mathomp4
Copy link
Copy Markdown
Contributor Author

I've merged develop into this PR since @tldahlgren merged in #1976. Now this is "pure" GFE as it should be :)

@mathomp4 mathomp4 requested a review from tclune October 15, 2025 16:05
@mathomp4
Copy link
Copy Markdown
Contributor Author

One note to @tldahlgren et al. The other maintainer of these codes is @tclune. Unfortunately, @tclune is a govt employee and so is on furlough. I did add him as a reviewer but he'll probably not see this. But both @climbfuji and @AlexanderRichert-NOAA have dealt with these enough that they are at least advanced builders :)

Copy link
Copy Markdown
Contributor

@AlexanderRichert-NOAA AlexanderRichert-NOAA left a comment

Choose a reason for hiding this comment

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

I'm still not solid on some of the new Spack logic as far as compiler dependencies, but as long as the Spack devs are fine with these dep definitions for pflogger and yafyaml then this is fine by me.

Copy link
Copy Markdown
Contributor

@tldahlgren tldahlgren left a comment

Choose a reason for hiding this comment

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

Confirmed the new version sha256s.

@tldahlgren tldahlgren merged commit 06bc6c6 into spack:develop Oct 16, 2025
17 checks passed
@mathomp4 mathomp4 deleted the gfe-update-2025Sep30 branch December 5, 2025 16:24
mkrack pushed a commit to mkrack/spack-packages that referenced this pull request Dec 23, 2025
* mapl: add commit hashes for esma_cmake resources

* gfe: Update GFE packages

* Fixes for ifx in spack

* Update to pFUnit v4.14.0

---------

Co-authored-by: Alex Richert <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants