Skip to contentSkip to conversation

Remove Bashisms from scripts/bitbake

PATCH New Joshua Lock 1 participant 2 messages Jan 31, 2011

lore
Share & tools
lore
#0 lore raw

cover Remove Bashisms from scripts/bitbake

From: Joshua Lock <[email protected]>

The BitBake wrapper script currently won't run under Dash, this patch removes the Bash only syntax and ensures the script will run under Dash so that users with this shell can get at least far enough to be advised that they should run Bash.

Pull URL: git://git.pokylinux.org/poky-contrib.git
Branch: josh/misc
Browse: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=josh/misc

Thanks,
Joshua Lock <[email protected]>
---

Joshua Lock (1):
scripts/bitbake: remove bashisms

 scripts/bitbake |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)
Signature · 1 line
1.7.3.5
#1 lore raw

PATCH 1/1 scripts/bitbake: remove bashisms

From: Joshua Lock <[email protected]>

As the BitBake script is the initial entry point for the system we need to ensure it can run in as many places as possible, including systems which aren't yest optimally configured for running Poky.
Remove some bashisms from the script so that it can run under Dash.

Pointers from: https://wiki.ubuntu.com/DashAsBinSh
Errors before this patch: http://pastie.org/1502136

---
 scripts/bitbake |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)
scripts/bitbake +6 -6
index 55bc8fd..f40bd28 100755
@@ -3,29 +3,29 @@ NO_BUILD_OPTS="--version -h --help -p --parse-only -s --show-versions -e --envir
 needpseudo="1"
 for opt in $@; do
 for key in $NO_BUILD_OPTS; do
-    if [ $opt == $key ]
+    if [ $opt = $key ]
     then
         needpseudo="0"
         break
     fi
 done
-[ $needpseudo == "0" ] && break
+[ $needpseudo = "0" ] && break
 done
 
 buildpseudo="1"
-if [ $needpseudo == "1" -a -e "$BUILDDIR/pseudodone" ]; then
+if [ $needpseudo = "1" ] && [ -e "$BUILDDIR/pseudodone" ]; then
     PSEUDOBINDIR=`cat $BUILDDIR/pseudodone`
     if [ -e "$PSEUDOBINDIR/pseudo" ]; then
         buildpseudo="0"
     fi
 fi
-if [ $needpseudo == "0" ]; then
+if [ $needpseudo = "0" ]; then
     buildpseudo="0"
 fi
 
 OLDPATH=$PATH
 export PATH=`echo $PATH | sed s#[^:]*/scripts:##`
-if [ $buildpseudo == "1" ]; then
+if [ $buildpseudo = "1" ]; then
     echo "Pseudo is not present but is required, building this first before the main build"
     bitbake pseudo-native -c populate_sysroot
     ret=$?
@@ -43,7 +43,7 @@ if [ $buildpseudo == "1" ]; then
 fi
 BITBAKE=`which bitbake`
 export PATH=$OLDPATH
-if [ $needpseudo == "1" ]; then
+if [ $needpseudo = "1" ]; then
     PSEUDOBINDIR=`cat $BUILDDIR/pseudodone`
     PSEUDO_BINDIR=$PSEUDOBINDIR PSEUDO_LIBDIR=$PSEUDOBINDIR/../lib/pseudo/lib PSEUDO_PREFIX=$PSEUDOBINDIR/../../ PSEUDO_DISABLED=1 $PSEUDOBINDIR/pseudo $BITBAKE $@
 else
Signature · 1 line
1.7.3.5