Skip to content
This repository was archived by the owner on Jun 30, 2021. It is now read-only.

Commit eeba60c

Browse files
christophegelgalu
authored andcommitted
Fix logs for video-recording (#184)
The log files "video-rec-stderr.log" and "video-rec-stdout.log" seem to contain "^M" characters (carriage-return). The 'tail' command below doesn't take into account this character. It returns the complete output of the log files. Therefore it needs to be replaced into a proper newline. This problem is not visible if executed in bash because each line will overwrite the previous line.
1 parent 999ca65 commit eeba60c

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

video-rec/bin/stop-video

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,9 @@ else
3939
die "Were unable to stop video recording within '${WAIT_TIME_OUT_VIDEO_STOP}'"
4040
fi
4141

42+
sed -i -e "s/\r/\n/g" ${LOGS_DIR}/video-rec-stderr.log || true
43+
sed -i -e "s/\r/\n/g" ${LOGS_DIR}/video-rec-stdout.log || true
44+
4245
# Include videos log in the output
4346
log "-- DEBUG: video-rec-stdout.log ----"
4447
tail -n 40 ${LOGS_DIR}/video-rec-stdout.log || true

0 commit comments

Comments
 (0)