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 #7

Merged
merged 1 commit into from
Apr 2, 2024

Conversation

gdicker1
Copy link
Contributor

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/ocn/source directory. Module use statements in the ocean source are changed from 'use mpas_...' to 'use mpaso_...' 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/ocn/source directory. Module use statements in the ocean
source are changed from 'use mpas_...' to 'use mpaso_...' to match
processing of the framework.
@gdicker1
Copy link
Contributor Author

@gdicker1
Copy link
Contributor Author

gdicker1 commented Apr 2, 2024

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.

(NOTE: This counts as my "review" since I can't self-request)

@gdicker1 gdicker1 merged commit b7d350a 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