Skip to content

Commit

Permalink
Merge pull request #3110 from branfosj/20240129170151_new_pr_comsol
Browse files Browse the repository at this point in the history
replace `run_cmd` with `run_shell_cmd` in custom easyblock for COMSOL (`comsol.py`)
  • Loading branch information
boegel authored Jan 30, 2024
2 parents e1f9765 + f9737dd commit e4576ed
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions easybuild/easyblocks/c/comsol.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
from easybuild.tools.build_log import EasyBuildError
from easybuild.tools.filetools import adjust_permissions, copy_file, find_flexlm_license, read_file, write_file
from easybuild.tools.modules import get_software_root
from easybuild.tools.run import run_cmd
from easybuild.tools.run import run_shell_cmd
from easybuild.tools.systemtools import get_shared_lib_ext


Expand Down Expand Up @@ -124,7 +124,7 @@ def install_step(self):
env.unset_env_vars(['DISPLAY'])

cmd = ' '.join([self.cfg['preinstallopts'], setup_script, '-s', self.configfile, self.cfg['installopts']])
run_cmd(cmd, log_all=True, simple=True)
run_shell_cmd(cmd)

def sanity_check_step(self):
"""Custom sanity check for COMSOL."""
Expand Down

0 comments on commit e4576ed

Please sign in to comment.