From 62d8453c2c743972c102dab250d4671a14860a56 Mon Sep 17 00:00:00 2001 From: Nathan Cutler Date: Wed, 1 Aug 2018 12:33:58 +0200 Subject: [PATCH] build/ops: unify command substitution in install-deps.sh The $() form is preferable to `` because folks (like me) might be using ` as a keyboard shortcut to GNU Screen, causing havoc to ensue whenever copy-pasting the ` character. Signed-off-by: Nathan Cutler (cherry picked from commit e0042dd617269934bf51efee5bab7e4485875576) --- install-deps.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/install-deps.sh b/install-deps.sh index c092a0a2cab7..18b6a8ea8103 100755 --- a/install-deps.sh +++ b/install-deps.sh @@ -21,7 +21,7 @@ if test $(id -u) != 0 ; then fi export LC_ALL=C # the following is vulnerable to i18n -ARCH=`uname -m` +ARCH=$(uname -m) function munge_ceph_spec_in { # http://rpm.org/user_doc/conditional_builds.html @@ -118,7 +118,7 @@ EOF fi } -if [ x`uname`x = xFreeBSDx ]; then +if [ x$(uname)x = xFreeBSDx ]; then $SUDO pkg install -yq \ devel/babeltrace \ devel/git \ -- 2.47.3