Skip to content

[release/public-v2.1.0] Modulefile updates for generic linux #478

Merged
MichaelLueken merged 1 commit into
ufs-community:release/public-v2.1.0from
natalie-perlin:ufs-srw-v2.1_linux
Nov 15, 2022
Merged

[release/public-v2.1.0] Modulefile updates for generic linux #478
MichaelLueken merged 1 commit into
ufs-community:release/public-v2.1.0from
natalie-perlin:ufs-srw-v2.1_linux

Conversation

@natalie-perlin
Copy link
Copy Markdown
Collaborator

Updated modulefiles: use srw_common module, conda environment instead of python
Build modulefile now uses srw_common module with list of standard software modules for the release.

The following files have been updated:
./modulefiles/build_linux_gnu.lua
./modulefiles/wflow_linux.lua

Type of change

  • Update of a supported feature (non-breaking change which adds functionality)
  • This change requires a documentation update
  • Similar change needed for develop at ufs-community/ufs-srweather-app

@MichaelLueken MichaelLueken changed the title Modulefile updates for generic linux [release/public-v2.1.0] Modulefile updates for generic linux Nov 15, 2022
@MichaelLueken MichaelLueken added the release This PR/issue is related to a release branch label Nov 15, 2022
Copy link
Copy Markdown
Collaborator

@danielabdi-noaa danielabdi-noaa left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Copy Markdown
Collaborator

@MichaelLueken MichaelLueken left a comment

Choose a reason for hiding this comment

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

These changes look good to me!

@MichaelLueken MichaelLueken merged commit 1dbdcf0 into ufs-community:release/public-v2.1.0 Nov 15, 2022
@natalie-perlin natalie-perlin deleted the ufs-srw-v2.1_linux branch October 13, 2023 04:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release This PR/issue is related to a release branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants