diff options
author | Leah Rowe <leah@libreboot.org> | 2023-09-09 23:15:17 +0100 |
---|---|---|
committer | Leah Rowe <leah@libreboot.org> | 2023-09-09 23:15:17 +0100 |
commit | 84bf47b5b947a8ed5afa1fb5335266d04ee3b004 (patch) | |
tree | 48e7512bececf2fc03f7d2eba4b45cd36bf219f4 /script/update/blobs/download | |
parent | b78009e28aab38bc149be5c57aa23505b8dd2a08 (diff) |
scripts: better handling of printf: stdout/stderr
in some cases, messages that should be considered errors
or warnings, were being written to the standard output,
rather than written as error messages.
also: one or two printf statements should specifically
avoid printing errors (to any file); in these cases,
stdout has been redirected to /dev/null
Signed-off-by: Leah Rowe <leah@libreboot.org>
Diffstat (limited to 'script/update/blobs/download')
-rwxr-xr-x | script/update/blobs/download | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/script/update/blobs/download b/script/update/blobs/download index b9c801a6..71c0cdb6 100755 --- a/script/update/blobs/download +++ b/script/update/blobs/download @@ -38,7 +38,8 @@ detect_firmware() [ "${CONFIG_INCLUDE_SMSC_SCH5545_EC_FW}" = "y" ] && \ needs="${needs} SCH5545EC" [ -z ${needs+x} ] && \ - printf "No binary blobs needed for this board\n" && \ + printf "No binary blobs needed for board: %s\n" "${board}" \ + 1>&2 && \ return 1 printf "Firmware needed for board '%s':\n%s\n" "${board}" "${needs}" } |