Skip to content

Commit 7bc1f0e

Browse files
authored
Merge pull request #4892 from boegel/exts_check_msg
replace full trace message for extension check command with simple pass/fail message
2 parents 3940f09 + 202d628 commit 7bc1f0e

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

easybuild/framework/extension.py

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@
4343
from easybuild.tools.build_log import EasyBuildError, EasyBuildExit, raise_nosupport
4444
from easybuild.tools.filetools import change_dir
4545
from easybuild.tools.run import run_shell_cmd
46+
from easybuild.tools.utilities import trace_msg
4647

4748

4849
def resolve_exts_filter_template(exts_filter, ext):
@@ -314,9 +315,13 @@ def sanity_check_step(self):
314315
self.log.info("modulename set to False for '%s' extension, so skipping sanity check", self.name)
315316
elif exts_filter:
316317
cmd, stdin = resolve_exts_filter_template(exts_filter, self)
317-
cmd_res = run_shell_cmd(cmd, fail_on_error=False, stdin=stdin)
318+
cmd_res = run_shell_cmd(cmd, fail_on_error=False, stdin=stdin, hidden=True)
318319

319-
if cmd_res.exit_code != EasyBuildExit.SUCCESS:
320+
msg = f"Extension sanity check command '{cmd}': "
321+
if cmd_res.exit_code == EasyBuildExit.SUCCESS:
322+
trace_msg(msg + 'OK')
323+
else:
324+
trace_msg(msg + 'FAIL')
320325
if stdin:
321326
fail_msg = 'command "%s" (stdin: "%s") failed' % (cmd, stdin)
322327
else:

0 commit comments

Comments
 (0)