take into account that path to file with command output may be None in cmd_trace_msg

boegel · boegel · commit 98a57205c6b9 · 2023-11-06T20:33:56.000+01:00
diff --git a/easybuild/tools/run.py b/easybuild/tools/run.py
@@ -348,7 +348,8 @@ def cmd_trace_msg(cmd, start_time, work_dir, stdin, cmd_out_fp, cmd_err_fp):
     ]
     if stdin:
         lines.append(f"\t[input: {stdin}]")
-    lines.append(f"\t[output saved to {cmd_out_fp}]")
+    if cmd_out_fp:
+        lines.append(f"\t[output saved to {cmd_out_fp}]")
     if cmd_err_fp:
         lines.append(f"\t[errors/warnings saved to {cmd_err_fp}]")
 

Original file line number	Diff line number	Diff line change
`@@ -348,7 +348,8 @@ def cmd_trace_msg(cmd, start_time, work_dir, stdin, cmd_out_fp, cmd_err_fp):`
`348`	`348`	`]`
`349`	`349`	`if stdin:`
`350`	`350`	`lines.append(f"\t[input: {stdin}]")`
`351`		`- lines.append(f"\t[output saved to {cmd_out_fp}]")`
	`351`	`+ if cmd_out_fp:`
	`352`	`+ lines.append(f"\t[output saved to {cmd_out_fp}]")`
`352`	`353`	`if cmd_err_fp:`
`353`	`354`	`lines.append(f"\t[errors/warnings saved to {cmd_err_fp}]")`
`354`	`355`