From: Philippe De Muyter <phdm@macqel.be>
Date: Fri, 6 Jun 2008 10:25:22 +0000 (+0200)
Subject: alsaconf:  use 'type -p', not which
X-Git-Tag: v1.0.17rc2~1
X-Git-Url: https://git.alsa-project.org/?a=commitdiff_plain;h=07ca0dd40b798c48cd55e1921e7fc6c4be3985ce;p=alsa-utils.git

alsaconf:  use 'type -p', not which

Here is a small patch for alsaconf, which removes an useless dependency
on `which' by using a bash built-in instead : `type -p'.  I encountered
the problem of the missing `which' while using alsaconf on a clfs-built
linux system.  It is useless to install `which' when we already have
`type -p', and alsaconf already needs bash, so this does not replace
a dependency by another one, but really removes a dependency for alsaconf.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
---

diff --git a/alsaconf/alsaconf.in b/alsaconf/alsaconf.in
index 123f488..f72f12b 100644
--- a/alsaconf/alsaconf.in
+++ b/alsaconf/alsaconf.in
@@ -31,7 +31,7 @@ PROCFS="/proc"
 SYSFS="/sys"
 
 # i18n stuff
-if test "$USE_NLS" = "yes" && which gettext > /dev/null; then
+if test "$USE_NLS" = "yes" && type -p gettext > /dev/null; then
   xecho() {
     gettext -s "$*"
   }
@@ -316,10 +316,10 @@ fi
 fi
 
 # Check for dialog, whiptail, gdialog, awk, ... ?
-if which dialog > /dev/null; then
+if type -p dialog > /dev/null; then
     DIALOG=dialog
 else
-  if which whiptail > /dev/null; then
+  if type -p whiptail > /dev/null; then
     whiptail_wrapper() {
       X1="$1"
       X2="$2"
@@ -337,7 +337,7 @@ else
     exit 1
   fi
 fi
-if which awk > /dev/null; then :
+if type -p awk > /dev/null; then :
 else
   xecho "Error, awk not found. Can't continue."
   exit 1