From 78260fb9e066ca6e391494b567b2758073491edc Mon Sep 17 00:00:00 2001 From: Chris Koeritz Date: Mon, 1 Jun 2020 17:02:20 -0400 Subject: [PATCH] settled on whichable func over direct call to which the whichable function does the which but also hides errors from not finding the command in question. this is much preferred over seeing errors about "gvim not found" etc. --- scripts/archival/compare_backup_drive.sh | 2 +- scripts/archival/list_arch.sh | 6 +++-- scripts/core/common.alias | 5 ++-- scripts/core/functions.sh | 32 ++++++++++++------------ scripts/core/variables.sh | 8 +++--- scripts/customize/fred/java_profile.sh | 4 +-- scripts/rev_control/version_control.sh | 4 +-- scripts/system/list_packages.sh | 4 +-- 8 files changed, 33 insertions(+), 32 deletions(-) diff --git a/scripts/archival/compare_backup_drive.sh b/scripts/archival/compare_backup_drive.sh index 501bfc97..4f36ceb4 100644 --- a/scripts/archival/compare_backup_drive.sh +++ b/scripts/archival/compare_backup_drive.sh @@ -22,7 +22,7 @@ echo would do-- compare_dirs "$target/$(basename $currdir)" "$currdir" # decide which drive to compare. targets="$1" if [ -z "$targets" ]; then - targets=($($(which ls) -1 /media/$USER/*)) + targets=($($(whichable ls) -1 /media/$USER/*)) if [ ${#targets[@]} -gt 1 ]; then echo " Please provide a media drive name on the command line, because more than diff --git a/scripts/archival/list_arch.sh b/scripts/archival/list_arch.sh index 2d5b6b1b..82e6fb7e 100644 --- a/scripts/archival/list_arch.sh +++ b/scripts/archival/list_arch.sh @@ -13,6 +13,8 @@ # An arbitrary format archive lister, although really we are mainly supporting # tar and zip currently, including compressed formats. +source "$FEISTY_MEOW_SCRIPTS/core/launch_feisty_meow.sh" + archive_file="$1"; shift if [ -z "$archive_file" ]; then echo "This script takes one archive name (in .tar.gz, .zip, etc. formats) and" @@ -25,9 +27,9 @@ if [ ! -f "$archive_file" ]; then fi if [ -z "$PAGER" ]; then - PAGER=$(which less) + PAGER=$(whichable less) if [ -z "$PAGER" ]; then - PAGER=$(which more) + PAGER=$(whichable more) if [ -z "$PAGER" ]; then PAGER="cat" fi diff --git a/scripts/core/common.alias b/scripts/core/common.alias index e62e2e31..5143f048 100644 --- a/scripts/core/common.alias +++ b/scripts/core/common.alias @@ -14,7 +14,6 @@ ############## # make our functions available to the aliases. -#hmmm: should this be necessary? will it even work? source "$FEISTY_MEOW_SCRIPTS/core/functions.sh" ############## @@ -47,11 +46,11 @@ if [ "$OS" != "Windows_NT" ]; then if [ -n "$IS_DARWIN" ]; then # case for mac os x. define_yeti_alias exp='open' - elif [ ! -z "$(which dolphin 2>/dev/null)" ]; then + elif [ ! -z "$(whichable dolphin)" ]; then # we prefer dolphin if available. define_yeti_alias exp='screen -L -S dolphin-$RANDOM -d -m dolphin ' #--select - elif [ ! -z "$(which nautilus 2>/dev/null)" ]; then + elif [ ! -z "$(whichable nautilus)" ]; then # launch nautilus if available. define_yeti_alias exp='nautilus' else diff --git a/scripts/core/functions.sh b/scripts/core/functions.sh index 3a2151b1..6989449f 100644 --- a/scripts/core/functions.sh +++ b/scripts/core/functions.sh @@ -33,6 +33,20 @@ if [ -z "$skip_all" ]; then date +"%Y$sep%m$sep%d$sep%H%M$sep%S" | tr -d '/\n/' } + # a wrapper for the which command that finds items on the path. some OSes + # do not provide which, so we want to not be spewing errors when that + # happens. + function whichable() + { + to_find="$1"; shift + local WHICHER="$(which which 2>/dev/null)" + if [ $? -ne 0 ]; then + # there is no which command here. we produce nothing due to this. + echo + fi + echo $($WHICHER $to_find) + } + # makes a directory of the name specified and then tries to change the # current directory to that directory. function mcd() { @@ -475,7 +489,7 @@ if [ -z "$skip_all" ]; then # overlay for nechung binary so that we can complain less grossly about it when it's missing. function nechung() { - local wheres_nechung=$(which nechung 2>/dev/null) + local wheres_nechung=$(whichable nechung) if [ -z "$wheres_nechung" ]; then echo "The nechung oracle program cannot be found. You may want to consider" echo "rebuilding the feisty meow applications with this command:" @@ -591,20 +605,6 @@ automatic if there is even a small amount of doubt about the issue." echo } - # a wrapper for the which command that finds items on the path. some OSes - # do not provide which, so we want to not be spewing errors when that - # happens. - function whichable() - { - to_find="$1"; shift - which which &>/dev/null - if [ $? -ne 0 ]; then - # there is no which command here. we produce nothing due to this. - echo - fi - echo $(which $to_find) - } - function add_cygwin_drive_mounts() { for i in c d e f g h q z ; do #hmmm: improve this by not adding the link if already there, or if the drive is not valid. @@ -860,7 +860,7 @@ return 0 this_host=$(hostname) elif [ ! -z "$(echo $MACHTYPE | grep suse)" ]; then this_host=$(hostname --long) - elif [ -x "$(which hostname 2>/dev/null)" ]; then + elif [ -x "$(whichable hostname)" ]; then this_host=$(hostname) fi echo "$this_host" diff --git a/scripts/core/variables.sh b/scripts/core/variables.sh index 8c21424f..7d13748e 100644 --- a/scripts/core/variables.sh +++ b/scripts/core/variables.sh @@ -331,20 +331,20 @@ export EDITOR # going to work well unless they can be prevented from forking the process # off. if [ -z "$EDITOR" ]; then - EDITOR="$(which gvim)" + EDITOR="$(whichable gvim)" if [ ! -z "$EDITOR" ]; then # if we found gvim, then add in the no forking flag. EDITOR+=" --nofork" fi fi if [ -z "$EDITOR" ]; then - EDITOR="$(which vim)" + EDITOR="$(whichable vim)" fi if [ -z "$EDITOR" ]; then - EDITOR="$(which vi)" + EDITOR="$(whichable vi)" fi if [ -z "$EDITOR" ]; then - EDITOR="$(which emacs)" + EDITOR="$(whichable emacs)" fi #### # out of ideas about editors at this point. diff --git a/scripts/customize/fred/java_profile.sh b/scripts/customize/fred/java_profile.sh index 99027e6f..b8e71a33 100644 --- a/scripts/customize/fred/java_profile.sh +++ b/scripts/customize/fred/java_profile.sh @@ -65,7 +65,7 @@ fi if [ ! -d "$JAVA_HOME" ]; then unset JAVA_HOME unset JAVA_BIN_PIECE - if [ -z "$(whichable java 2>/dev/null)" ]; then + if [ -z "$(whichable java)" ]; then intuition_failure JAVA_HOME fi fi @@ -104,7 +104,7 @@ else ECLIPSE_DIR=$(echo $ECLIPSE_DIR | sed -e 's/^\(.\):/\/cygdrive\/\1/') fi fi -if [ -z "$ECLIPSE_DIR" -a -z "$(whichable eclipse 2>/dev/null)" ]; then +if [ -z "$ECLIPSE_DIR" -a -z "$(whichable eclipse)" ]; then intuition_failure ECLIPSE_DIR fi diff --git a/scripts/rev_control/version_control.sh b/scripts/rev_control/version_control.sh index 5342ef2c..703e003d 100644 --- a/scripts/rev_control/version_control.sh +++ b/scripts/rev_control/version_control.sh @@ -15,8 +15,8 @@ source "$FEISTY_MEOW_SCRIPTS/tty/terminal_titler.sh" export MAX_DEPTH=5 # use our splitter tool for lengthy output if it's available. -if [ ! -z "$(which splitter 2>/dev/null)" ]; then - TO_SPLITTER="$(which splitter)" +if [ ! -z "$(whichable splitter)" ]; then + TO_SPLITTER="$(whichable splitter)" # calculate the number of columsn in the terminal. cols=$(get_maxcols) TO_SPLITTER+=" --maxcol $(($cols - 1))" diff --git a/scripts/system/list_packages.sh b/scripts/system/list_packages.sh index 60024c75..0a52aa64 100644 --- a/scripts/system/list_packages.sh +++ b/scripts/system/list_packages.sh @@ -16,14 +16,14 @@ if debian_like; then exit $? fi -rpm_available="$(which rpm)" +rpm_available="$(whichable rpm)" if [ ! -z "$rpm_available" ]; then #is that the right phrase for rpm? somewhat forgotten. rpm -qa | eval $SEEK_PIECE exit $? fi -yum_available="$(which yum)" +yum_available="$(whichable yum)" if [ ! -z "$yum_available" ]; then yum list | eval $SEEK_PIECE exit $? -- 2.34.1