Skip to content

Commit

Permalink
build/ops: unify command substitution in install-deps.sh
Browse files Browse the repository at this point in the history
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 <[email protected]>
  • Loading branch information
smithfarm committed Aug 1, 2018
1 parent f170775 commit e0042dd
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions install-deps.sh
Original file line number Diff line number Diff line change
Expand Up @@ -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 install_seastar_deps {
if [ $WITH_SEASTAR ]; then
Expand Down Expand Up @@ -135,7 +135,7 @@ EOF
fi
}

if [ x`uname`x = xFreeBSDx ]; then
if [ x$(uname)x = xFreeBSDx ]; then
$SUDO pkg install -yq \
devel/babeltrace \
devel/git \
Expand Down

0 comments on commit e0042dd

Please sign in to comment.