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

Removes dlapqc.f from sources built into libglimmer. Fixes #85. #86

Merged

Conversation

jeff-cohere
Copy link
Contributor

dlapqc.f contains a program, which produces a duplicate "main" symbol that prevents a successful build of the glimmer-cism library on Macs. This change simply removes that file from the list of sources to be built into the library.

[BFB]

dlapqc.f contains a program, which produces a duplicate "main" symbol
that prevents a successful build of the glimmer-cism library on Macs.
@jeff-cohere
Copy link
Contributor Author

@rljacob, any idea who should handle this?

@douglasjacobsen
Copy link
Member

@jnjohnsonlbl That should be one of the CISM developers. I'll talk to people around here and see if I can get someone to handle it for you.

@jeff-cohere
Copy link
Contributor Author

Thanks, Doug.

@matthewhoffman
Copy link
Contributor

I've confirmed that this module has been removed from the build in CISM 2.0, so I feel comfortable saying it is ok to remove dlapqc.f from the CISM build in ACME.

@douglasjacobsen , can you handle the merge?

@douglasjacobsen
Copy link
Member

@matthewhoffman Thanks. I'll handle it now.

@douglasjacobsen douglasjacobsen merged commit c25412e into master Jan 28, 2015
@douglasjacobsen douglasjacobsen deleted the jnjohnsonlbl/models/fix_glimmer_cism_build_on_mac branch January 28, 2015 18:49
jgfouca pushed a commit that referenced this pull request Jun 25, 2019
update the intel compiler for cheyenne
yunpengshan2014 pushed a commit that referenced this pull request Apr 2, 2024
amametjanov pushed a commit to amametjanov/E3SM that referenced this pull request Jun 5, 2024
…mmon

Common test functionality and operators refactor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants