-
Notifications
You must be signed in to change notification settings - Fork 13
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
Rename the amsua_metop-c CRTM-fix file on copying #399
Rename the amsua_metop-c CRTM-fix file on copying #399
Conversation
@DavidHuber-NOAA Have you tested this PR? |
@climbfuji Yes, I tested a before and after. The correct amsua_metop-c fix file is copied over and all other fix files are identical. |
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.
Thanks! Let's wait for @AlexanderRichert-NOAA to review this before I merge. We also need to find a way to backport this into the 1.6.0 release (I can take care of that).
@AlexanderRichert-NOAA Can you please review this PR? |
This causes spack to copy the erroneously-placed fix file SpcCoeff/Little_Endian/amsua_metop-c_v2.SpcCoeff.bin when copying big-endian fix files (i.e. +big-endian) to the final fix directory as amsua_metop-c.SpcCoeff.bin.
Description
This causes spack to copy the erroneously-placed fix file
SpcCoeff/Little_Endian/amsua_metop-c_v2.SpcCoeff.bin
when copying big-endian fix files (i.e.+big-endian
) to the finalfix
directory asamsua_metop-c.SpcCoeff.bin
.Issue(s) addressed
Resolves #963
Checklist