From: Abongwa Bonalais Amahnui <abongwabonalais@gmail.com>
To: bitbake-devel@lists.openembedded.org
Cc: Abongwa Bonalais Amahnui <abongwabonalais@gmail.com>
Subject: [PATCH] toaster: Add quoting of variables to avoid splitting
Date: Tue, 5 Apr 2022 22:41:00 +0100 [thread overview]
Message-ID: <20220405214100.61887-1-abongwabonalais@gmail.com> (raw)
Signed-off-by: Abongwa Bonalais Amahnui <abongwabonalais@gmail.com>
---
bitbake/bin/toaster | 66 ++++++++++++++++++++++-----------------------
1 file changed, 33 insertions(+), 33 deletions(-)
diff --git a/bitbake/bin/toaster b/bitbake/bin/toaster
index 558a819570..e190f2c9db 100755
--- a/bitbake/bin/toaster
+++ b/bitbake/bin/toaster
@@ -19,9 +19,9 @@ Usage 2: source toaster manage [createsuperuser|lsupdates|migrate|makemigrations
custom_extention()
{
- custom_extension=$BBBASEDIR/lib/toaster/orm/fixtures/custom_toaster_append.sh
- if [ -f $custom_extension ] ; then
- $custom_extension $*
+ custom_extension="$BBBASEDIR/lib/toaster/orm/fixtures/custom_toaster_append.sh"
+ if [ -f "$custom_extension" ] ; then
+ $custom_extension "$*"
fi
}
@@ -51,19 +51,19 @@ databaseCheck()
webserverKillAll()
{
local pidfile
- if [ -f ${BUILDDIR}/.toastermain.pid ] ; then
+ if [ -f "${BUILDDIR}/.toastermain.pid" ] ; then
custom_extention web_stop_postpend
else
custom_extention noweb_stop_postpend
fi
- for pidfile in ${BUILDDIR}/.toastermain.pid ${BUILDDIR}/.runbuilds.pid; do
- if [ -f ${pidfile} ]; then
- pid=`cat ${pidfile}`
- while kill -0 $pid 2>/dev/null; do
- kill -SIGTERM $pid 2>/dev/null
+ for pidfile in "${BUILDDIR}/.toastermain.pid" "${BUILDDIR}/.runbuilds.pid"; do
+ if [ -f "${pidfile}" ]; then
+ pid=`cat "${pidfile}"`
+ while kill -0 "$pid" 2>/dev/null; do
+ kill -SIGTERM "$pid" 2>/dev/null
sleep 1
done
- rm ${pidfile}
+ rm "${pidfile}"
fi
done
}
@@ -84,8 +84,8 @@ webserverStartAll()
echo "Starting webserver..."
$MANAGE runserver --noreload "$ADDR_PORT" \
- </dev/null >>${BUILDDIR}/toaster_web.log 2>&1 \
- & echo $! >${BUILDDIR}/.toastermain.pid
+ </dev/null >>"${BUILDDIR}/toaster_web.log" 2>&1 \
+ & echo "$!" >"${BUILDDIR}/.toastermain.pid"
sleep 1
@@ -95,7 +95,7 @@ webserverStartAll()
else
echo "Toaster development webserver started at http://$ADDR_PORT"
echo -e "\nYou can now run 'bitbake <target>' on the command line and monitor your build in Toaster.\nYou can also use a Toaster project to configure and run a build.\n"
- custom_extention web_start_postpend $ADDR_PORT
+ custom_extention web_start_postpend "$ADDR_PORT"
fi
return $retval
@@ -131,8 +131,8 @@ verify_prereq() {
exp=$exp'vmax=["%02d" % int(n) for n in "\4".split(".")];'
exp=$exp'sys.exit(not (version \1 vmin and version \3 vmax))'
exp=$exp'/p'
- if ! sed -n "$exp" $reqfile | python3 - ; then
- req=`grep ^Django $reqfile`
+ if ! sed -n "$exp" "$reqfile" | python3 - ; then
+ req=`grep ^Django "$reqfile"`
echo "This program needs $req"
echo "Please install with pip3 install -r $reqfile"
return 2
@@ -150,10 +150,10 @@ else
TOASTER=$0
fi
-export BBBASEDIR=`dirname $TOASTER`/..
+export BBBASEDIR=`dirname "$TOASTER"`/..
MANAGE="python3 $BBBASEDIR/lib/toaster/manage.py"
if [ -z "$OE_ROOT" ]; then
- OE_ROOT=`dirname $TOASTER`/../..
+ OE_ROOT=`dirname "$TOASTER"`/../..
fi
# this is the configuraton file we are using for toaster
@@ -164,7 +164,7 @@ fi
# in the local layers that currently make using an arbitrary
# toasterconf.json difficult.
-. $OE_ROOT/.templateconf
+. "$OE_ROOT/.templateconf"
if [ -n "$TEMPLATECONF" ]; then
if [ ! -d "$TEMPLATECONF" ]; then
# Allow TEMPLATECONF=meta-xyz/conf as a shortcut
@@ -199,10 +199,10 @@ for param in $*; do
webport=*)
ADDR_PORT="${param#*=}"
# Split the addr:port string
- ADDR=`echo $ADDR_PORT | cut -f 1 -d ':'`
- PORT=`echo $ADDR_PORT | cut -f 2 -d ':'`
+ ADDR=`echo "$ADDR_PORT" | cut -f 1 -d ':'`
+ PORT=`echo "$ADDR_PORT" | cut -f 2 -d ':'`
# If only a port has been speified then set address to localhost.
- if [ $ADDR = $PORT ] ; then
+ if [ "$ADDR" = "$PORT" ] ; then
ADDR_PORT="localhost:$PORT"
fi
;;
@@ -229,7 +229,7 @@ for param in $*; do
esac
done
-if [ `basename \"$0\"` = `basename \"${TOASTER}\"` ]; then
+if [ `basename "$0"` = `basename "${TOASTER}"` ]; then
echo "Error: This script needs to be sourced. Please run as . $TOASTER"
return 1
fi
@@ -265,7 +265,7 @@ fi
echo "The system will $CMD."
# Execute the commands
-custom_extention toaster_prepend $CMD $ADDR_PORT
+custom_extention toaster_prepend "$CMD" "$ADDR_PORT"
case $CMD in
start )
@@ -277,9 +277,9 @@ case $CMD in
fi
# Create configuration file
- conf=${BUILDDIR}/conf/local.conf
+ conf="${BUILDDIR}/conf/local.conf"
line='INHERIT+="toaster buildhistory"'
- grep -q "$line" $conf || echo $line >> $conf
+ grep -q "$line" "$conf" || echo "$line" >> "$conf"
if [ $WEBSERVER -eq 0 ] ; then
# Do not update the database for "noweb" unless
@@ -290,17 +290,17 @@ case $CMD in
return 4
fi
fi
- custom_extention noweb_start_postpend $ADDR_PORT
+ custom_extention noweb_start_postpend "$ADDR_PORT"
fi
- if [ $WEBSERVER -gt 0 ] && ! webserverStartAll; then
+ if [ "$WEBSERVER" -gt 0 ] && ! webserverStartAll; then
echo "Failed ${CMD}."
return 4
fi
export BITBAKE_UI='toasterui'
if [ $TOASTER_BUILDSERVER -eq 1 ] ; then
$MANAGE runbuilds \
- </dev/null >>${BUILDDIR}/toaster_runbuilds.log 2>&1 \
- & echo $! >${BUILDDIR}/.runbuilds.pid
+ </dev/null >>"${BUILDDIR}/toaster_runbuilds.log" 2>&1 \
+ & echo $! >"${BUILDDIR}/.runbuilds.pid"
else
echo "Toaster build server not started."
fi
@@ -308,7 +308,7 @@ case $CMD in
# set fail safe stop system on terminal exit
trap stop_system SIGHUP
echo "Successful ${CMD}."
- custom_extention toaster_postpend $CMD $ADDR_PORT
+ custom_extention toaster_postpend "$CMD" "$ADDR_PORT"
return 0
;;
stop )
@@ -316,9 +316,9 @@ case $CMD in
echo "Successful ${CMD}."
;;
manage )
- cd $BBBASEDIR/lib/toaster
- $MANAGE $manage_cmd
+ cd "$BBBASEDIR/lib/toaster"
+ "$MANAGE $manage_cmd"
;;
esac
-custom_extention toaster_postpend $CMD $ADDR_PORT
+custom_extention toaster_postpend "$CMD" "$ADDR_PORT"
--
2.25.1
next reply other threads:[~2022-04-06 16:33 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-05 21:41 Abongwa Bonalais Amahnui [this message]
2022-04-05 21:47 [PATCH] toaster: Add quoting of variables to avoid splitting Abongwa Bonalais Amahnui
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220405214100.61887-1-abongwabonalais@gmail.com \
--to=abongwabonalais@gmail.com \
--cc=bitbake-devel@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.