Skip to content

Commit

Permalink
Merge pull request #1703 from ESMCI/sarich/scripts/fix-skip-preview-n…
Browse files Browse the repository at this point in the history
…amelist-parsing-order

Fix bug in case.run where arg skip-preview-namelist is out of sequence

arg parsing needs to be done after adding arguments
master is broken on acme mira/cetus and blues/anvil without this fix

Test suite: acme_developer on anvil
Test baseline:
Test namelist changes:
Test status:

Fixes #1690

User interface changes?:

Update gh-pages html (Y/N)?:

Code review:
  • Loading branch information
jgfouca authored Jun 26, 2017
2 parents 15c229f + b732931 commit 4b116ba
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion config/acme/machines/template.case.run
Original file line number Diff line number Diff line change
Expand Up @@ -51,11 +51,12 @@ OR
parser.add_argument("--caseroot",
help="Case directory to build")

args = CIME.utils.parse_args_and_handle_standard_logging_options(args, parser)

parser.add_argument("--skip-preview-namelist", action="store_true",
help="Skip calling preview-namelist during case.run")

args = CIME.utils.parse_args_and_handle_standard_logging_options(args, parser)

if args.caseroot is not None:
os.chdir(args.caseroot)

Expand Down

0 comments on commit 4b116ba

Please sign in to comment.