Skip to content

Update CCPP framework and physics CMakeLists.txt; forward ccpp-framework to HEAD of main #940

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

Merged
merged 6 commits into from
Apr 16, 2025

Conversation

@climbfuji climbfuji changed the title Update CCPP framework and physics CMakeLists.txt; forward framework to HEAD of main Update CCPP framework and physics CMakeLists.txt; forward ccpp-framework to HEAD of main Mar 21, 2025
@climbfuji climbfuji self-assigned this Mar 21, 2025
@climbfuji climbfuji marked this pull request as ready for review March 21, 2025 11:46
grantfirl added a commit to NCAR/ccpp-framework that referenced this pull request Apr 16, 2025
…an modules in the correct place (#653)

**Update src/CMakeLists.txt: use modern cmake features to install
Fortran modules in the correct place**

Update CMakeLists.txt to modern cmake version 3 and support NEPTUNE
cmake build while retaining compatibility with UFS/SCM

User interface changes?: No

Fixes: [Github issue #s] n/a

Testing: all CI tests pass, ran `capgen` and `prebuild` tests locally on
my laptop - all pass
  test removed:
  unit tests:
  system tests:
  manual testing:

Associated PRs:
- ccpp-physics: ufs-community/ccpp-physics#259
- FV3: NOAA-EMC/fv3atm#940
- ufs-weather-model:
ufs-community/ufs-weather-model#2659
@grantfirl
Copy link
Collaborator

@climbfuji @jkbk2004 The associated CCPP PRs have been merged.

@climbfuji
Copy link
Collaborator Author

working on it

@climbfuji
Copy link
Collaborator Author

@jkbk2004 @grantfirl done

@jkbk2004 jkbk2004 merged commit 463686c into NOAA-EMC:develop Apr 16, 2025
3 of 8 checks passed
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.

5 participants