diff options
| author | Andrew Robbins <contact@andrewrobbins.info> | 2017-10-26 21:45:42 -0400 |
|---|---|---|
| committer | Andrew Robbins <contact@andrewrobbins.info> | 2017-10-28 23:30:03 -0400 |
| commit | 5c1fe562d8044249f5830df826544bc72ecb41b2 (patch) | |
| tree | e86f14dfddc867e8a8abdfa31713edcc2cf87c3d /libs/common | |
| parent | c7377e3eb74a73874f9d2253083b3ae952156ac8 (diff) | |
| download | librebootfr-5c1fe562d8044249f5830df826544bc72ecb41b2.tar.gz librebootfr-5c1fe562d8044249f5830df826544bc72ecb41b2.zip | |
Remove unnecessary spaces in command substitutions
This is likely one of the very last changes necessary to make the
Libreboot build system more cohesive in appearance.
Hopefully from this point forward it won't be as readily apparent as
to who wrote which parts of the build system (i.e. won't look like a
patchwork quilt any longer).
Diffstat (limited to 'libs/common')
| -rwxr-xr-x | libs/common | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/libs/common b/libs/common index 28a34cf7..43fe3b08 100755 --- a/libs/common +++ b/libs/common @@ -130,7 +130,7 @@ path_wildcard_expand() { local path=$@ # Evaluation fails with unescaped whitespaces. - path=$( printf '%s\n' "$path" | sed "s/ /\\\ /g" ) + path=$(printf '%s\n' "$path" | sed "s/ /\\\ /g") eval "arguments_list "$path"" } @@ -139,8 +139,8 @@ file_checksum_create() { local path=$1 local checksum_path="$path.$CHECKSUM" - local name=$( basename "$path" ) - local directory_path=$( dirname "$path" ) + local name=$(basename "$path") + local directory_path=$(dirname "$path") ( cd "$directory_path" @@ -152,8 +152,8 @@ file_checksum_check() { local path=$1 local checksum_path="$path.$CHECKSUM" - local name=$( basename "$path" ) - local directory_path=$( dirname "$path" ) + local name=$(basename "$path") + local directory_path=$(dirname "$path") if ! [[ -f "$checksum_path" ]] then @@ -217,7 +217,7 @@ file_exists_check() { directory_filled_check() { local path=$1 - if [[ -z "$( ls -A "$path" 2> /dev/null )" ]] + if [[ -z "$(ls -A "$path" 2> /dev/null)" ]] then return 1 else @@ -379,7 +379,7 @@ requirements() { for requirement in "$@" do - requirement_path=$( which "$requirement" || true ) + requirement_path=$(which "$requirement" || true) if [[ -z "$requirement_path" ]] then @@ -396,7 +396,7 @@ requirements_root() { for requirement in "$@" do # We need to keep stdout output to show the command. - requirement_path=$( execute_root which "$requirement" || true ) + requirement_path=$(execute_root which "$requirement" || true) if [[ -z "$requirement_path" ]] then @@ -426,7 +426,7 @@ arguments_concat() { } execute_root() { - local sudo=$( which sudo 2> /dev/null || true ) + local sudo=$(which sudo 2> /dev/null || true) local arguments printf 1>&2 '%s' 'Running command as root: ' @@ -437,7 +437,7 @@ execute_root() { sudo "$@" else # Quote arguments for eval through su. - arguments=$( printf '%q ' "$@" ) + arguments=$(printf '%q ' "$@") su -c "$arguments" fi } |