From 6af65ad430ae58bac5afd0d6e12a97e5e12e9b59 Mon Sep 17 00:00:00 2001 From: Leah Rowe Date: Fri, 20 Oct 2023 04:10:50 +0100 Subject: error handling code cleanup and fixes in some cases, use of x_ or xx_ can be error-prone, due to the way $@ is handled; commands requiring quotes, or with funny file names as arguments such as spaces in the file name, or other special characters, can make the x/xx functions break. in those cases, where x/xx must not be used, the commands use || err instead in other cases, use of x/xx is superfluous, and has been removed in some commands. Signed-off-by: Leah Rowe --- include/option.sh | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'include/option.sh') diff --git a/include/option.sh b/include/option.sh index dd41e88c..13137104 100755 --- a/include/option.sh +++ b/include/option.sh @@ -67,8 +67,7 @@ handle_coreboot_utils() x_ mkdir -p "cbutils/${1}" && \ x_ cp "src/coreboot/${1}/util/${util}/${util}" \ "cbutils/${1}" - [ -z "${mode}" ] || \ - x_ rm -Rf "cbutils/${1}" + [ -z "${mode}" ] || x_ rm -Rf "cbutils/${1}" done } -- cgit v1.2.1