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

Elimination of conflicting external references #9

Merged
merged 2 commits into from
Apr 2, 2024

Conversation

dazlich
Copy link
Contributor

@dazlich dazlich commented Mar 9, 2024

These modifications allow for the elimination of conflicting external references at the link step of compilation. They go hand in hand with branches of the same name in mpas-framework and mpas-seaice. The buildlib now has some sed statements that replace a dummy string through the framework files and rename them in the bld/ice/source directory. Module use statements in the ocean source are changed from 'use mpas_...' to 'use mpass_...' to match processing of the framework.

external references at the link step of compilation. They go
hand in hand with branches of the same name in mpas-framework
and mpas-seaice. The buildlib now has some sed statements that
replace a dummy string through the framework files and rename them
in the bld/ice/source directory. Module use statements in the ocean
source are changed from 'use mpas_...' to 'use mpass_...' to match
processing of the framework.
@dazlich
Copy link
Contributor Author

dazlich commented Mar 9, 2024

Looks like the previous PR slipped a cime_config file in. How do I eliminate it?

@dazlich
Copy link
Contributor Author

dazlich commented Mar 12, 2024

I've got the conflicts worked out, and the prescribe seaice mode changes in the develop branch are now merge in, so review away @gdicker1 @areanddee @sherimickelson @supreethms1809

@gdicker1
Copy link
Contributor

@gdicker1
Copy link
Contributor

Looks like the previous PR slipped a cime_config file in. How do I eliminate it?

I think when I tested merging this PR into ew-develop it resolved itself. I'll make a follow-up PR to add some more things to the .gitignore so files like *.pyc or *.swp don't get included.

@dazlich
Copy link
Contributor Author

dazlich commented Mar 29, 2024 via email

Copy link
Contributor

@gdicker1 gdicker1 left a comment

Choose a reason for hiding this comment

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

I now think my issue with the term "external reference" in the commit message isn't worth these changes waiting any longer. Without further discussion or thoughts, I plan to merge these PRs and EWOrg/EW #39 later this afternoon.

@gdicker1 gdicker1 merged commit 3e6e8d2 into ew-develop Apr 2, 2024
@gdicker1 gdicker1 deleted the framework-ext-refs branch April 23, 2024 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants