From 84bf47b5b947a8ed5afa1fb5335266d04ee3b004 Mon Sep 17 00:00:00 2001 From: Leah Rowe Date: Sat, 9 Sep 2023 23:15:17 +0100 Subject: 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 --- script/build/command/options | 2 +- script/build/release/roms | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'script/build') diff --git a/script/build/command/options b/script/build/command/options index 84a9a3fb..dd22078f 100755 --- a/script/build/command/options +++ b/script/build/command/options @@ -22,7 +22,7 @@ listitems() # -e used because this is for files *or* directories [ -e "${x}" ] || continue [ "${x##*/}" = "build.list" ] && continue - printf "%s\n" "${x##*/}" + printf "%s\n" "${x##*/}" 2>/dev/null items=0 done return ${items} diff --git a/script/build/release/roms b/script/build/release/roms index c551c1aa..da67f8ed 100755 --- a/script/build/release/roms +++ b/script/build/release/roms @@ -191,9 +191,9 @@ strip_ucode() "${cbfstool}" "${_newrom_b}" remove -n \ cpu_microcode_blob.bin 2>/dev/null || microcode_present="n" if [ "${microcode_present}" = "n" ]; then - printf "REMARK: '%s' already lacks microcode\n" \ - "${romfile}" - printf "Renaming default ROM file instead.\n" + printf "REMARK: '%s' already lacks microcode\n" "${romfile}" \ + 1>&2 + printf "Renaming default ROM file instead.\n" 1>&2 mv "${romfile}" "${_newrom_b}" || \ err "strip_rom_image: can't rename no-u ${romfile}" fi -- cgit v1.2.1