-
Notifications
You must be signed in to change notification settings - Fork 176
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
Refactor some build scripts #794
Merged
WalterKolczynski-NOAA
merged 6 commits into
NOAA-EMC:develop
from
WalterKolczynski-NOAA:feature/build_app
May 18, 2022
Merged
Changes from 1 commit
Commits
Show all changes
6 commits
Select commit
Hold shift + click to select a range
ede3ebe
Refactor some build scripts
WalterKolczynski-NOAA 198f501
Fix indentation in build usage messages
WalterKolczynski-NOAA b6243ee
Merge branch 'develop' into feature/build_app
WalterKolczynski-NOAA e661b57
Remove debug print and ensure bash
WalterKolczynski-NOAA 1afd787
Restore missing indentation in partial_build
WalterKolczynski-NOAA a3ad198
Remove option to set CCPP suites from command line
WalterKolczynski-NOAA File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next
Next commit
Refactor some build scripts
Refactors the build_ufs script to allow the build of any UFS app. Additionally, the user may now supply a list of CCPP suites from the command line. The new format of the command is now: build_ufs.sh [-a UFS_app][-s CCPP_suites][-v] -a UFS_app: Specify the UFS application to build. The default if none is provided is S2SWA -s CCPP_suites: Build UFS with given comma-separated list of CCPP suites instead of default list -v: Turn on verbose mode (BUILD_VERBOSE) build_all.sh is similarly refactored to include the same three options. Also, a new '-c build_config' option is added to specify an alternative list of programs to build. build_all.sh [-a UFS_app][-c build_config][-h][-s CCPP_SUITES][-v] -a UFS_app: Build a specific UFS app instead of the default -c build_config: Selectively build based on the provided config instead of the default config -h: Print usage message and exit -s CCPP_suites: Build UFS with given comma-separated list of CCPP suites instead of default list -v: Run all scripts in verbose mode partial_build.sh is also updated to take the new -c option to specify a build config. partial_build.sh [-c config_file][-h][-v] -c config_file: Selectively build based on the provided config. The default if none is specified is gfs_build.cfg -h: Print usage message and exit -v: Run in verbose mode In addition to the above, build_all and partial_build had their indentation redone and their usage/help statement updated/added. Also, the build config- uration file was renamed to from fv3gfs_build.cfg to gfs_build.cfg. Fixes #745 Fixes #751
- Loading branch information
commit ede3ebe1e865cb61ab4c5ad052e27e555db56ff7
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does
set -eu
work? If so, please add.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, missed this before I merged. But adding
set -eu
to all scripts is part of my long-term plans (see Issue #397).