Skip to content

Commit

Permalink
check-sof-logger: change BEG>> to BEG:: to avoid HTML conflict
Browse files Browse the repository at this point in the history
Also change <<END to ::END and print both files always.

<< and >> seem to corrupt the generated HTML and I could not find
anything parsing these anyway as of now.

See discussion in (much larger) PR thesofproject#666 for more details.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
  • Loading branch information
marc-hb committed Apr 27, 2021
1 parent c31f270 commit 6a42ab5
Showing 1 changed file with 9 additions and 5 deletions.
14 changes: 9 additions & 5 deletions test-case/check-sof-logger.sh
Original file line number Diff line number Diff line change
Expand Up @@ -56,18 +56,22 @@ sudo pkill -9 "$(basename "$loggerBin")" 2> /dev/null

func_logger_exit()
{
local code=$1 type=${2:-data}
dlogi "Log $type BEG>>"
cat "$LOG_ROOT/logger.$type.log"
dlogi "<<END $type data"
local code=$1

for ftype in data error; do
printf '\n'
dlogi "Log $ftype BEG::"
cat "$LOG_ROOT/logger.$ftype.log"
dlogi "::END $ftype"
done
exit "$code"
}

# check if we get any sof-logger errors
logger_err=$(grep -i 'error' "$error_file")
if [[ $logger_err ]]; then
dloge "No available log to export due to sof-logger errors."
func_logger_exit 1 'error'
func_logger_exit 1
fi

# '\.c\:[1-9]' to filter like '.c:6' this type keyword like:
Expand Down

0 comments on commit 6a42ab5

Please sign in to comment.