diff options
author | Masahiro Yamada <yamada.m@jp.panasonic.com> | 2015-02-03 19:14:46 +0900 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2015-02-04 12:51:22 +0100 |
commit | a93f9429765d7995016c8073ba03cbfa01ad5996 (patch) | |
tree | dd567f3eaeb062bf34bba0a5d3717a56e45d3498 /MAKEALL | |
parent | 7aa78c3709e3ee656300bcbfda8b4b6353b182fd (diff) | |
download | barebox-a93f9429765d7995016c8073ba03cbfa01ad5996.tar.gz barebox-a93f9429765d7995016c8073ba03cbfa01ad5996.tar.xz |
MAKEALL: use $(...) instead of `...` for readability
The mixture of single quotes and back quotes in the same line makes
scripts unreadable.
Here, do_build_target() is especially unreadable. Using $(...)
instead of `...` would make it a little better.
Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'MAKEALL')
-rwxr-xr-x | MAKEALL | 14 |
1 files changed, 7 insertions, 7 deletions
@@ -14,7 +14,7 @@ here=$(pwd) time_start=$(date +%s) -filename=`basename $0` +filename=$(basename $0) is_print_stats=1 @@ -97,14 +97,14 @@ do_build_target() { mkdir -p "${LOGDIR}/${target}" printf "Building ${arch} ${target} \n" >&2 | tee -a "${log_report}" - tmp=`echo "${target}" | tr - _` + tmp=$(echo "${target}" | tr - _) - cross_compile=`eval echo '$CROSS_COMPILE_'${tmp}` - cross_compile_set=`eval echo '${CROSS_COMPILE_'${tmp}'+set}'` + cross_compile=$(eval echo '$CROSS_COMPILE_'${tmp}) + cross_compile_set=$(eval echo '${CROSS_COMPILE_'${tmp}'+set}') if [ "${cross_compile_set}" = "" ] then - cross_compile=`eval echo '$CROSS_COMPILE_'${arch}` - cross_compile_set=`eval echo '${CROSS_COMPILE_'${arch}'+set}'` + cross_compile=$(eval echo '$CROSS_COMPILE_'${arch}) + cross_compile_set=$(eval echo '${CROSS_COMPILE_'${arch}'+set}') if [ "${cross_compile_set}" = "" ] then cross_compile=${CROSS_COMPILE} @@ -239,7 +239,7 @@ fi if [ "${CONFIG}" ] then - basedir=`dirname ${CONFIG}` + basedir=$(dirname ${CONFIG}) if [ ! "${basedir}" ] || [ "${basedir}" = "." ] then |