Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/56056 )
Change subject: util/board_status: Do not display grep message ......................................................................
util/board_status: Do not display grep message
Redirect stdout and stderr from grep to check for unknown timestamps, when no timestamps are stored, which is already logged earlier.
Failed to run "/root/coreboot/util/cbmem/cbmem -t", ignoring Getting remote dmesg grep: /tmp/coreboot_board_status.dXmbUIBP/emulation/qemu-i440fx/4.14-876-gdb28040ee1/2021-07-02T23_14_33Z/coreboot_timestamps.txt: No such file or directory
Change-Id: Ib5400d4bd17e957b4cc1bf75bbd332d60ad226f5 Signed-off-by: Paul Menzel pmenzel@molgen.mpg.de Reviewed-on: https://review.coreboot.org/c/coreboot/+/56056 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Angel Pons th3fanbus@gmail.com --- M util/board_status/board_status.sh 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Angel Pons: Looks good to me, approved
diff --git a/util/board_status/board_status.sh b/util/board_status/board_status.sh index 292bffa..f5254f1 100755 --- a/util/board_status/board_status.sh +++ b/util/board_status/board_status.sh @@ -501,7 +501,7 @@ exit $EXIT_FAILURE fi
-if [ $(grep -- unknown "${tmpdir}/${results}/coreboot_timestamps.txt") ]; then +if [ $(grep -- unknown "${tmpdir}/${results}/coreboot_timestamps.txt" >/dev/null 2>&1) ]; then echo "Unknown timestamps found in 'coreboot_timestamps.txt'." \ "Please rebuild the 'cbmem' utility and try again." exit $EXIT_FAILURE