Skip to content

Commit d57c6a4

Browse files
Stop depending on $BASETMPDIR for os::cmd tempfiles
The resolution of the temporary file assignment in the `os::cmd` code now happens before `$BASETMPDIR` is set, so we should not rely on it. Signed-off-by: Steve Kuznetsov <[email protected]>
1 parent 3da3a2b commit d57c6a4

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

hack/lib/cmd.sh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -157,8 +157,7 @@ readonly -f os::cmd::try_until_text
157157

158158
# In order to harvest stderr and stdout at the same time into different buckets, we need to stick them into files
159159
# in an intermediate step
160-
BASETMPDIR="${TMPDIR:-"/tmp"}/openshift"
161-
os_cmd_internal_tmpdir="${BASETMPDIR}/test-cmd"
160+
os_cmd_internal_tmpdir="${TMPDIR:-"/tmp"}/openshift"
162161
os_cmd_internal_tmpout="${os_cmd_internal_tmpdir}/tmp_stdout.log"
163162
os_cmd_internal_tmperr="${os_cmd_internal_tmpdir}/tmp_stderr.log"
164163

0 commit comments

Comments
 (0)