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

Generalize and consolidate build system for porting to a new platform #176

Closed
jwolff-ncar opened this issue Mar 23, 2020 · 5 comments
Closed
Assignees
Milestone

Comments

@jwolff-ncar
Copy link
Contributor

People should not have to modify dozens of different files in different places to port to their system, nor should they have to do so to add support for a different compiler or other type of different configuration. Setting up module files should also not be a requirement.

  • Config file for different platforms

  • Centralized location

  • Build on what global did (Dom, Laurie)

    • For global: One configuration for libraries (incl pre/post), CIME has a config file (for model)
  • Clean script to run before recompile

@jwolff-ncar
Copy link
Contributor Author

@mkavulich Can you update this issue with information from the release build focus team? Should we close this issue or link it to other open issues that are related to keep tabs on progress?

@mkavulich
Copy link
Collaborator

This will be achieved once the top-level cmake build is in place. This is waiting on UFS_UTILS PR#118 (ufs-community/UFS_UTILS#118) so that the latest NCEPLIBS can be used for the entire app, as well as Larissa's upcoming PR that will allow us to use the authoritative UFS_UTILS repository. The cmake build is in this fork, and a PR will be opened once all the UFS_UTILS changes are in place.

@jwolff-ncar
Copy link
Contributor Author

@mkavulich @llpcarson Is there another issue this should be linked to that is being actively worked on? Or does this just need to be updated?

@mkavulich
Copy link
Collaborator

The top-level cmake build is now in place; the last thing before it is finalized will be this PR for NCEPLIBS which creates files that set up the user's environment for building the app. See that PR for more details: NOAA-EMC/NCEPLIBS#148

@mkavulich
Copy link
Collaborator

The cmake build is now platform-generic. Users should be able to install NCEPLIBS and point to it with cmake, and this should theoretically work on any machine provided NCEPLIBS is installed successfully and the proper environment variables are set.

christinaholtNOAA pushed a commit to christinaholtNOAA/regional_workflow that referenced this issue Aug 26, 2021
* Add NWGES space to save all boundary, initial, restart files.
Each cycle will search for nwges directory for boundary, initial and restart files.
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

No branches or pull requests

5 participants