Skip to content

Commit

Permalink
Merge pull request #3144 from branfosj/20240203094556_new_pr_star_ccm
Browse files Browse the repository at this point in the history
replace `run_cmd` with `run_shell_cmd` in custom easyblock for STAR-CCM+ (`star_ccm.py`)
  • Loading branch information
boegel authored Feb 11, 2024
2 parents 16ef96d + 9611626 commit 5f099de
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions easybuild/easyblocks/s/star_ccm.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
from easybuild.framework.easyblock import EasyBlock
from easybuild.tools.config import build_option
from easybuild.tools.filetools import change_dir, find_glob_pattern
from easybuild.tools.run import run_cmd
from easybuild.tools.run import run_shell_cmd


class EB_STAR_minus_CCM_plus_(EasyBlock):
Expand Down Expand Up @@ -81,7 +81,7 @@ def install_step(self):

# ignore exit code of command, since there's always a non-zero exit if $CHECK_DISK_SPACE is set to OFF;
# rely on sanity check to catch problems with the installation
run_cmd(cmd, log_all=False, log_ok=False, simple=False)
run_shell_cmd(cmd, fail_on_error=False)

def find_starccm_subdirs(self):
"""Determine subdirectory of install directory in which STAR-CCM+ was installed."""
Expand Down

0 comments on commit 5f099de

Please sign in to comment.