Skip to content

Commit

Permalink
Merge Pull Request #2768 from E3SM-Project/scream/bartgol/eamxx/use-p…
Browse files Browse the repository at this point in the history
…netcdf-in-tas

Automatically Merged using E3SM Pull Request AutoTester
PR Title: Add pnetcdf paths to config line when running test-all-scream
PR Author: bartgol
PR LABELS: testing, AT: AUTOMERGE, scripts, AT: Skip v1 Testing
  • Loading branch information
E3SM-Bot authored Mar 21, 2024
2 parents 815e4cd + b75ebea commit 66b6f25
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion components/eamxx/scripts/test_all_scream.py
Original file line number Diff line number Diff line change
Expand Up @@ -443,6 +443,9 @@ def generate_cmake_config(self, test, for_ctest=False):
stat, c_path, _ = run_cmd("nc-config --prefix")
if stat == 0:
result += f" -DNetCDF_C_PATH={c_path}"
stat, pc_path, _ = run_cmd("pnetcdf-config --prefix")
if stat == 0:
result += f" -DPnetCDF_C_PATH={pc_path}"

# Test-specific cmake options
for key, value in test.cmake_args:
Expand Down Expand Up @@ -865,6 +868,7 @@ def test_all_scream(self):

finally:
# Cleanup the repo if needed
cleanup_repo(self._original_branch, self._original_commit, self._has_backup_commit)
if self._original_commit!=get_current_commit():
cleanup_repo(self._original_branch, self._original_commit, self._has_backup_commit)

return success

0 comments on commit 66b6f25

Please sign in to comment.