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.

Sign up for GitHub

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

Jump to bottom

Remove deprecated file config_compilers.xml #35

Open
jedwards4b opened this issue May 4, 2022 · 3 comments
Open

Remove deprecated file config_compilers.xml #35

jedwards4b opened this issue May 4, 2022 · 3 comments
Assignees
@jedwards4b

Comments

@jedwards4b
Copy link
Collaborator

Does anyone have a reason to keep this file around, it is already creating confusion in the user community. Perhaps if we still need it we can add a banner at the top indicating that the file is
deprecated.
@jedwards4b jedwards4b self-assigned this May 4, 2022
@jedwards4b jedwards4b changed the title Remove depricated file config_compilers.xml Remove deprecated file config_compilers.xml May 4, 2022
@sjsprecious
Copy link
Collaborator

Hi @jedwards4b , do I understand it correctly that everything set in the config_compilers.xml is now set under machines/cmake_macros/*cmake files?

@jedwards4b
Copy link
Collaborator Author

Yes that is correct.

@sjsprecious
Copy link
Collaborator

Thanks @jedwards4b. Maybe it is better to just remove this file rather than put a banner to avoid confusion, as long as the CESM code builds correctly?

mvertens added a commit to mvertens/ccs_config_cesm that referenced this issue Apr 1, 2024
@mvertens
Merge pull request ESMCI#35 from mvertens/feature/fix_ww3_grids
7ab13d4 
fixes and additions to ww3 grids
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jedwards4b jedwards4b

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jedwards4b @sjsprecious