Skip to content

Lfric warnings wp #82

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

Merged
merged 2 commits into from
Oct 27, 2023
Merged

Lfric warnings wp #82

merged 2 commits into from
Oct 27, 2023

Conversation

alejandrobodas
Copy link
Collaborator

@alejandrobodas alejandrobodas commented Oct 25, 2023

lfric builds do not specify the Fortran standard to which the program is expected to conform. Unfortunately, not setting a -stand or -std flag in builds has allowed some non-standard code to make its way onto the socrates trunk. This PR fixes a couple of wp missing.

@alejandrobodas
Copy link
Collaborator Author

Hi @dustinswales , would you mind doing another one of these small ones? Thanks!

Copy link
Contributor

@dustinswales dustinswales left a comment

Choose a reason for hiding this comment

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

@alejandrobodas Thanks for doing this cleanup!

@alejandrobodas alejandrobodas merged commit c60fbc6 into master Oct 27, 2023
@alejandrobodas
Copy link
Collaborator Author

Thanks @dustinswales , merging the changes

@alejandrobodas alejandrobodas deleted the lfric_warnings_wp branch October 27, 2023 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants