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

revert jcsda modifications to lib/.../modules/* #400

Conversation

AlexanderRichert-NOAA
Copy link
Collaborator

@AlexanderRichert-NOAA AlexanderRichert-NOAA commented Jan 26, 2024

Description

Remove our customizations to Spack's module logic. See JCSDA/spack-stack#972 for details & discussion.

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

@AlexanderRichert-NOAA AlexanderRichert-NOAA marked this pull request as ready for review January 27, 2024 01:04
@climbfuji
Copy link
Collaborator

@AlexanderRichert-NOAA I updated your branch from jcsda_emc_spack_stack. I also created a PR that you will need to merge/pull in so that this reversal of the lowercase package_ROOT env variables works: AlexanderRichert-NOAA#2

Replace geos_ROOT with GEOS_ROOT in var/spack/repos/builtin/packages/py-shapely/shapely-1.8.0-geos.py.patch
Copy link
Collaborator

@climbfuji climbfuji left a comment

Choose a reason for hiding this comment

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

This does require minor changes in JEDI (jedi-bundle and repos underneath), but these are trivial to make. Happy to see this reverted, thanks for doing this.

@climbfuji
Copy link
Collaborator

Note that this also requires changes in GEOS-GCM, see GEOS-ESM/GEOSgcm#767

@climbfuji climbfuji merged commit 3839d67 into JCSDA:jcsda_emc_spack_stack Mar 7, 2024
13 checks passed
@climbfuji climbfuji deleted the revert_modules_mods branch March 7, 2024 22:34
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