Skip to content

Conversation

@jim-p-w
Copy link
Contributor

@jim-p-w jim-p-w commented May 15, 2025

This change updates the core_init_atmosphere cmake file to add new files added in commit d2c0680 (PR #1276).
That PR updated the makefiles, but not the cmake files. That resulted in compile errors when building with cmake:
src/core_init_atmosphere/mpas_init_atm_cases.F:48:11:

48 | use mpas_init_atm_gwd_gsl, only : calc_gsl_oro_data
| 1
Fatal Error: Cannot open module file 'mpas_init_atm_gwd_gsl.mod' for reading at
(1): No such file or directory

Tested by building mpas_bundle.

added in commit d2c0680  (PR MPAS-Dev#1276).
That PR updated the makefiles, but not the cmake files. That resulted in
compile errors when building with cmake:
src/core_init_atmosphere/mpas_init_atm_cases.F:48:11:

   48 |       use mpas_init_atm_gwd_gsl, only : calc_gsl_oro_data
      |           1
Fatal Error: Cannot open module file 'mpas_init_atm_gwd_gsl.mod' for reading at
(1): No such file or directory
@mgduda mgduda requested review from amstokely and mgduda May 15, 2025 23:17
@mgduda
Copy link
Contributor

mgduda commented May 15, 2025

@jim-p-w Is the branch name for this PR the same as in #1316? If so, it might be preferable to use a new branch name for this PR, perhaps init_atmosphere/cmake_noaa_ugwp. The branch name ends up in the first line of a merge commit message, so we'd otherwise end up with these two lines in the commit log for the develop branch:

Merge branch 'atmosphere/cmake_noaa_ugwp' into develop (PR #1316)

and

Merge branch 'atmosphere/cmake_noaa_ugwp' into develop (PR #1317)

which might be confusing.

@jim-p-w
Copy link
Contributor Author

jim-p-w commented May 16, 2025

@mgduda I don't see how to change the source branch for this PR. It appears only the target branch can be changed. I think I will need to create a new PR and close this one?

@jim-p-w
Copy link
Contributor Author

jim-p-w commented May 16, 2025

This PR is superseded by #1320.

@jim-p-w jim-p-w closed this May 16, 2025
@jim-p-w jim-p-w deleted the atmosphere/cmake_noaa_ugwp branch May 28, 2025 20:08
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.

3 participants