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

ccpp_prebuild: ccpp stub broken due to hardcoded name of ccpp_t variable #496

Closed
climbfuji opened this issue Sep 7, 2023 · 0 comments · Fixed by #498
Closed

ccpp_prebuild: ccpp stub broken due to hardcoded name of ccpp_t variable #496

climbfuji opened this issue Sep 7, 2023 · 0 comments · Fixed by #498
Assignees
Labels

Comments

@climbfuji
Copy link
Collaborator

Description

Turns out the CCPP stub in directory stub is currently broken in main, following an earlier PR.

The problem is that the ccpp_prebuild.py code uses a hardcoded name for the CCPP data (type ccpp_t) variable, which is ok as long as every host model calls this variable cdata itself - not a good and safe assumption. Parts of the code already use the actual name of the variable as it is provided by the host model, but not all do.

Steps to Reproduce

Check out ccpp-framework main and follow the directions in stub/README.md.

Additional Context

n/a

Output

n/a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
1 participant