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 which to find ncdump and an error message if not found #4418

Merged
merged 1 commit into from
Aug 1, 2023

Conversation

jedwards4b
Copy link
Contributor

Use shutils.which to find ncdump, issue an error if it's not in the path.

Test suite: ERR.f19_g17.X.cheyenne_intel
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes #4173

User interface changes?:

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

@jedwards4b jedwards4b requested a review from briandobbins May 4, 2023 13:25
@jedwards4b jedwards4b self-assigned this May 4, 2023
@jedwards4b jedwards4b merged commit c6dd05d into ESMCI:master Aug 1, 2023
@jedwards4b jedwards4b deleted the use_which_ncdump branch August 1, 2023 14:43
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.

Issue running short-term archiver when 'ncdump' is not in the PATH
1 participant