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

fix issue with configure and mpilib #2591

Merged
merged 1 commit into from
May 14, 2018

Conversation

jedwards4b
Copy link
Contributor

@jedwards4b jedwards4b commented May 14, 2018

Fixes an issue when selecting mpilib on cheyenne
Test suite: tools/configure --mpilib mpt
Test baseline:
Test namelist changes:
Test status: [bit for bit, roundoff, climate changing]

Fixes an issue raised in #2589

User interface changes?:

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

Code review:

Copy link
Contributor

@mnlevy1981 mnlevy1981 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good (and introduces yet another error into the ESMF toolkit discussion)

@mnlevy1981 mnlevy1981 merged commit 0b07cb5 into ESMCI:master May 14, 2018
jgfouca added a commit that referenced this pull request May 18, 2018
* esmci_remote_for_split/master: (473 commits)
  add commas to output using --value
  remove debug print and traceback
  fix multisubmit tests/case2 was not run
  fixes PET test
  fix formatting issue
  these grids are needed for cime standalone testing
  response to review
  better fix for xmlquery
  Better error message on cheyenne login node
  Parallel map generation on cheyenne
  set even thought it isnt used
  update iyear for 2000 runs
  Update ChangeLog to cime5.6.7
  Better README files for mapping tools
  update readme
  PR #2591 allows a little more code clean-up
  update readme
  dont show variables not found unless --debug is used
  add model_version
  fix issue with configure and mpilib
  ...
@jedwards4b jedwards4b deleted the configure_mpilib branch June 6, 2019 20:42
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