-
Notifications
You must be signed in to change notification settings - Fork 20
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
Build on Gaea #67
Build on Gaea #67
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I only have 1 comment that holds this PR back.
It is the same comment I had in #64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
Thanks for adding the capability to build on Gaea.
@aerorahul Morning..The epic software team got back to me yesterday and had a small correction to the gaea module file for gfs_utils..It doesn't change anything but eliminates a python version number warning message. I have committed that change and rebuilt. What's the process to correct this PR? New PR for bug fix? Reopen here? Thanks |
Yes. A new PR with a hotfix/bugfix is generally how this is handled. |
Great. Will do. Thanks. |
Description
Add/update module file scripts for compiling gfs_utils on Gaea-C5
Refs #66
Type of change
Change characteristics
How has this been tested?
Checklist