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

preview_run: Fix case where mpirun cmd uses env vars #3139

Merged
merged 1 commit into from
Aug 16, 2019

Conversation

jgfouca
Copy link
Member

@jgfouca jgfouca commented Aug 16, 2019

The mpirun cmd was being retrieved before the full environment was
being loaded, leading to unresolved env vars appearing in the command.

[BFB]

The mpirun cmd was being retrieved before the full environment was
being loaded, leading to unresolved env vars appearing in the command.

[BFB]
@jgfouca jgfouca added Scripts bug fix PR BFB PR leaves answers BFB CIME labels Aug 16, 2019
@jgfouca jgfouca self-assigned this Aug 16, 2019
jgfouca added a commit that referenced this pull request Aug 16, 2019
preview_run: Fix case where mpirun cmd uses env vars

The mpirun cmd was being retrieved before the full environment was
being loaded, leading to unresolved env vars appearing in the command.

[BFB]

* origin/jgfouca/cime/fix_prev_run_env_vars:
  preview_run: Fix case where mpirun cmd uses env vars
  Update CIME to ESMCI cime5.8.7-2 (PR #3126)
jgfouca added a commit that referenced this pull request Aug 16, 2019
preview_run: Fix case where mpirun cmd uses env vars

The mpirun cmd was being retrieved before the full environment was
being loaded, leading to unresolved env vars appearing in the command.

[BFB]

* origin/jgfouca/cime/fix_prev_run_env_vars:
  preview_run: Fix case where mpirun cmd uses env vars
  Update CIME to ESMCI cime5.8.7-2 (PR #3126)
@jgfouca jgfouca merged commit 6c8ef68 into master Aug 16, 2019
@jgfouca jgfouca deleted the jgfouca/cime/fix_prev_run_env_vars branch August 16, 2019 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant