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

Add --nowrf option to configure script, for configuring without WRF #175

Merged
merged 1 commit into from
May 7, 2021

Conversation

mgduda
Copy link
Collaborator

@mgduda mgduda commented May 5, 2021

This PR adds a new option, --nowrf, to the configure script so that the WPS
can be configured without a compiled WRF model.

Only three of the WPS components -- geogrid, metgrid, and int2nc -- require
the WRF I/O libraries. Even if none of these components was needed, it was
previously still necessary to either compile WRF or to make manual changes to
the configure script.

This PR introduces logic in the configure script to check for a command-
line argument, --nowrf, that, if present, disables checks in the configure
script for a compiled WRF directory. This enables, for example, the compilation
of the ungrib program without needing to compile the WRF model.

Only three of the WPS components -- geogrid, metgrid, and int2nc -- require
the WRF I/O libraries. Even if none of these components was needed, it was
previously still necessary to either compile WRF or to make manual changes to
the configure script.

This commit introduces logic in the configure script to check for a command-
line argument, --nowrf, that, if present, disables checks in the configure
script for a compiled WRF directory. This enables, for example, the compilation
of the ungrib program without needing to compile the WRF model.
@weiwangncar
Copy link
Collaborator

@mgduda I tested it on Cheyenne with and without --nowrf, and it works as it is supposed to. I will approve this PR.

@mgduda mgduda merged commit 56a3e8b into wrf-model:develop May 7, 2021
@mgduda mgduda deleted the configure_no_wrf branch May 7, 2021 03:54
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.

2 participants