summaryrefslogtreecommitdiff
path: root/include/option.sh
diff options
context:
space:
mode:
authorLeah Rowe <leah@libreboot.org>2023-10-07 05:36:52 +0100
committerLeah Rowe <leah@libreboot.org>2023-10-07 05:37:11 +0100
commit4708da2ca9a72520f39749b3b34b0bdc2fbfa15d (patch)
tree8c7f436fc53d10665f150e295d2d488b3f86e729 /include/option.sh
parent0fad3497b82d9225468835d1b27717050ca6de46 (diff)
use quotes when checking empty strings in scripts
this is far less error-prone Signed-off-by: Leah Rowe <leah@libreboot.org>
Diffstat (limited to 'include/option.sh')
-rwxr-xr-xinclude/option.sh4
1 files changed, 2 insertions, 2 deletions
diff --git a/include/option.sh b/include/option.sh
index 6ae5f2de..bc5bb42a 100755
--- a/include/option.sh
+++ b/include/option.sh
@@ -41,11 +41,11 @@ handle_coreboot_utils()
{
for util in cbfstool ifdtool; do
x_ ./update project build ${_f} "src/coreboot/${1}/util/${util}"
- [ -z ${mode} ] && [ ! -f "cbutils/${1}/${util}" ] && \
+ [ -z "${mode}" ] && [ ! -f "cbutils/${1}/${util}" ] && \
x_ mkdir -p "cbutils/${1}" && \
x_ cp "src/coreboot/${1}/util/${util}/${util}" \
"cbutils/${1}"
- [ -z ${mode} ] || \
+ [ -z "${mode}" ] || \
x_ rm -Rf "cbutils/${1}"
done
}