Skip to content

Updates from macOS testing#2003

Draft
mathomp4 wants to merge 43 commits intoJCSDA:developfrom
GMAO-SI-Team:geos-testing
Draft

Updates from macOS testing#2003
mathomp4 wants to merge 43 commits intoJCSDA:developfrom
GMAO-SI-Team:geos-testing

Conversation

@mathomp4
Copy link
Copy Markdown
Collaborator

@mathomp4 mathomp4 commented Apr 28, 2026

Description

This has updates needed in my testing on macOS.

One change that is "generic" to many is the change to configs/common/modules_lmod.yaml and configs/common/modules_tcl.yaml, e.g.,:

      python:
        environment:
          set:
            'Python_ROOT_DIR': '{prefix}'
            'Python3_ROOT_DIR': '{prefix}'

The reason is because CMake's FindPython and FindPython3 look at Python_ROOT_DIR and Python3_ROOT_DIR as hints, but spack only provides python_ROOT_DIR...and CMake is case-sensitive.

This also has a change to allow both older and newer FMS as GEOS (at present) only works with FMS 2024.

Dependencies

JCSDA/spack-packages#53

Issues addressed

Closes #1983
Closes #1982
Closes #1984

Applications affected

None

Systems affected

None

Testing

  • CI: Note whether the automatic tests (GitHub actions tests that run automatically for every commit) pass or not
    • GitHub actions CI tests pass
    • GitHub actions CI tests do not pass (provide explanation)
    • GitHub actions CI tests skipped (provide explanation if necessary)
  • New tests added: List and describe any new tests added to GitHub actions
    • ...
  • Additional testing: Add information on any additional tests conducted
    • ...

Checklist

  • This PR addresses one issue/problem/enhancement or has a very good reason for not doing so.
  • These changes have been tested on the affected systems and applications.
  • All dependency PRs/issues have been resolved and this PR can be merged.
  • All necessary updates to the documentation (spack-stack wiki) will be made when this PR is merged

@mathomp4 mathomp4 changed the title WIP: Updates from macOS testing Updates from macOS testing May 4, 2026
@climbfuji
Copy link
Copy Markdown
Collaborator

@mathomp4 Can you resolve the conflicts, please? It would be nice to test this + JCSDA/spack-packages#53 in CI. Thanks!

@mathomp4
Copy link
Copy Markdown
Collaborator Author

mathomp4 commented May 6, 2026

@mathomp4 Can you resolve the conflicts, please? It would be nice to test this + JCSDA/spack-packages#53 in CI. Thanks!

@climbfuji I think I did this right? I had to lean on some GPT to help because it was a new git error/warning I'd never seen before (since it involved a submodule, etc)

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.

[INSTALL]: libiconv "latest" [INSTALL]: openmpi 5.0.10 [INSTALL]: openblas 0.3.32

2 participants