From 2d9ab1f7f82f9a98b97d1503c1e3f31c86061c15 Mon Sep 17 00:00:00 2001 From: johnraff Date: Thu, 12 Mar 2020 14:37:21 +0900 Subject: Test for executables: replace 'which' with more robust 'command -v' Instances of: if [ $(which ] have been replaced with: if command -v >/dev/null which is considered to be more robust in a range of environments. scripts/build/chroot_archives: line 259: if [ "${LB_APT}" = "aptitude" ] && [ ! $(Chroot chroot "which aptitude") ] has been left untouched because the chroot might require a more complex command which would need more testing. manpages/Makefile: line 42: @if [ ! -x "$$(which po4a 2>/dev/null)" ]; \ has been left untouched because I am not sufficiently familiar with makefiles. --- scripts/build/bootstrap_debootstrap | 2 +- scripts/build/chroot_archives | 2 +- scripts/build/config | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) (limited to 'scripts/build') diff --git a/scripts/build/bootstrap_debootstrap b/scripts/build/bootstrap_debootstrap index c25cbacec..d1fe54365 100755 --- a/scripts/build/bootstrap_debootstrap +++ b/scripts/build/bootstrap_debootstrap @@ -21,7 +21,7 @@ USAGE="${PROGRAM} [--force]" # Processing arguments and configuration files Init_config_data "${@}" -if [ ! $(which debootstrap) ] +if ! command -v debootstrap >/dev/null then Echo_error "debootstrap - command not found" Echo_verbose "debootstrap can be obtained from http://ftp.debian.org/debian/pool/main/d/debootstrap/" diff --git a/scripts/build/chroot_archives b/scripts/build/chroot_archives index b602caf4b..612f8f74a 100755 --- a/scripts/build/chroot_archives +++ b/scripts/build/chroot_archives @@ -238,7 +238,7 @@ case "${1}" in cp -f cache/indices.bootstrap/*Release* chroot/var/lib/apt/lists fi - if [ "${LB_APT}" = "aptitude" ] && [ ! $(which aptitude) ] + if [ "${LB_APT}" = "aptitude" ] && ! command -v aptitude >/dev/null then Chroot chroot "apt-get ${APT_OPTIONS} update" Chroot chroot "apt-get ${APT_OPTIONS} install aptitude" diff --git a/scripts/build/config b/scripts/build/config index 0f84c28c7..943ca7c00 100755 --- a/scripts/build/config +++ b/scripts/build/config @@ -182,7 +182,7 @@ Local_arguments () case "${1}" in --dump) # Dump version - if [ $(which dpkg-query) ] + if command -v dpkg-query >/dev/null then VERSION_DPKG="$(dpkg-query -f='${Version}' --show live-build)" fi -- cgit v1.2.3