Skip to content

moninshoc.meta missing mfpbl.f dependency#934

Merged
SamuelTrahanNOAA merged 1 commit into
NCAR:mainfrom
SamuelTrahanNOAA:bugfix/sam-broke-moorthi
Jun 17, 2022
Merged

moninshoc.meta missing mfpbl.f dependency#934
SamuelTrahanNOAA merged 1 commit into
NCAR:mainfrom
SamuelTrahanNOAA:bugfix/sam-broke-moorthi

Conversation

@SamuelTrahanNOAA
Copy link
Copy Markdown
Contributor

@SamuelTrahanNOAA SamuelTrahanNOAA commented Jun 6, 2022

A recent addition of "module" statements at the top of some files broke moninshoc. This adds a dependency so moninshoc will compile.

Fixes #935

@ligiabernardet
Copy link
Copy Markdown
Collaborator

@SamuelTrahanNOAA Does this affect the release branch and the suites planned for release as part of CCPP v6?

@SamuelTrahanNOAA
Copy link
Copy Markdown
Contributor Author

Only if the release branch has JM's Neptune changes.

@ligiabernardet
Copy link
Copy Markdown
Collaborator

The release does not have JM's NEPTUNE changes. Thanks for the prompt reply.

@grantfirl
Copy link
Copy Markdown
Collaborator

@SamuelTrahanNOAA I realize that this PR doesn't change the baseline and #942 does, but since this is just one line, could this be combined into #942 for the sake of saving UFS code managers work?

@SamuelTrahanNOAA
Copy link
Copy Markdown
Contributor Author

This has been merged into #942

@SamuelTrahanNOAA SamuelTrahanNOAA merged commit 3aee79f into NCAR:main Jun 17, 2022
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.

moninshoc fails to compile because of missing mfpbl.f dependency

4 participants