Browse Source

debian-bootstrap.pre: Modern shell scripting (use $() instead of ``)

Various shell files contain a mix between obsolete ``
and modern $(); It would be nice to convert to using $()
everywhere.

Signed-off-by: Mao Zhongyi <maozhongyi@cmss.chinamobile.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Tested-by: Alex Bennée <alex.bennee@linaro.org>
Signed-off-by: Thomas Huth <thuth@redhat.com>
Mao Zhongyi 6 years ago
parent
commit
b91068cd34
1 changed files with 2 additions and 2 deletions
  1. 2 2
      tests/docker/dockerfiles/debian-bootstrap.pre

+ 2 - 2
tests/docker/dockerfiles/debian-bootstrap.pre

@@ -2,7 +2,7 @@
 #
 #
 # Simple wrapper for debootstrap, run in the docker build context
 # Simple wrapper for debootstrap, run in the docker build context
 #
 #
-FAKEROOT=`which fakeroot 2> /dev/null`
+FAKEROOT=$(which fakeroot 2> /dev/null)
 # debootstrap < 1.0.67 generates empty sources.list, see Debian#732255
 # debootstrap < 1.0.67 generates empty sources.list, see Debian#732255
 MIN_DEBOOTSTRAP_VERSION=1.0.67
 MIN_DEBOOTSTRAP_VERSION=1.0.67
 
 
@@ -52,7 +52,7 @@ fi
 
 
 if [ -z $DEBOOTSTRAP_DIR ]; then
 if [ -z $DEBOOTSTRAP_DIR ]; then
     NEED_DEBOOTSTRAP=false
     NEED_DEBOOTSTRAP=false
-    DEBOOTSTRAP=`which debootstrap 2> /dev/null`
+    DEBOOTSTRAP=$(which debootstrap 2> /dev/null)
     if [ -z $DEBOOTSTRAP ]; then
     if [ -z $DEBOOTSTRAP ]; then
         echo "No debootstrap installed, attempting to install from SCM"
         echo "No debootstrap installed, attempting to install from SCM"
         NEED_DEBOOTSTRAP=true
         NEED_DEBOOTSTRAP=true