From 5e4e9b2cea4d391296570846c6bef81b2da7309f Mon Sep 17 00:00:00 2001
From: Chris Koeritz <fred@gruntose.com>
Date: Sun, 29 May 2022 08:17:35 -0400
Subject: [PATCH] updated for mp3s

needed to split the implementation for basic sound files vs. mp3s.  only mplayer is supported for mp3 playback on command line currently.
---
 scripts/multimedia/sound_play.sh | 48 ++++++++++++++++++++++----------
 1 file changed, 34 insertions(+), 14 deletions(-)

diff --git a/scripts/multimedia/sound_play.sh b/scripts/multimedia/sound_play.sh
index 8c5130ef..250bddb4 100644
--- a/scripts/multimedia/sound_play.sh
+++ b/scripts/multimedia/sound_play.sh
@@ -9,33 +9,53 @@ if [ $# -lt 1 ]; then
   exit 0;
 fi
 
-export PLAYCMD=/usr/bin/play
-if [ ! -f "$PLAYCMD" ]; then
-  PLAYCMD='echo Unknown sound player...'
-fi
+export BASIC_PLAY_CMD='echo Unknown basic sound player...'
 
-if [ ! -z "$(whichable afplay)" ]; then
+if [ -f "/usr/bin/play" ]; then
+#echo we see /usr/bin/play available...
+  BASIC_PLAY_CMD=/usr/bin/play
+elif [ ! -z "$(whichable afplay)" ]; then
 #echo we see afplay available...
-  PLAYCMD=afplay
+  BASIC_PLAY_CMD=afplay
 elif [ ! -z "$(psfind artsd)" ]; then
 #echo we see artsd running...
-  PLAYCMD=artsplay
+  BASIC_PLAY_CMD=artsplay
 elif [ ! -z "$(psfind esd)" ]; then
 #echo  we see esd running...
-  PLAYCMD=esdplay
+  BASIC_PLAY_CMD=esdplay
 elif [ ! -z "$(psfind pulseaudio)" ]; then
 #echo we see pulse running...
-  PLAYCMD="padsp aplay"
+  BASIC_PLAY_CMD="padsp aplay"
 elif [ ! -z "$WINDIR" ]; then
 #echo kludge for win32; we provide our own sound player.
-  PLAYCMD=playsound
+  BASIC_PLAY_CMD=playsound
+else
+  echo "I don't know how to play basic sound files for this OS and sound system."
+fi
+
+export MP3_PLAY_CMD='echo Unknown mp3 player...'
+
+if [ ! -z "$(whichable mplayer)" ]; then
+  MP3_PLAY_CMD=mplayer
 else
-  echo "I don't know how to play sounds for this OS and sound system."
+  echo "I don't know how to play mp3 files for this OS and sound system."
 fi
 
-# play the sounds individually; playsound can handle multiple files, but
-# "play" doesn't want to on some systems.
-for i in $*; do $PLAYCMD $i >/dev/null 2>&1; done
+# play the sounds individually; some apps like playsound can handle multiple
+# files, but "/usr/bin/play" doesn't want to on some systems.
+for filename in $*; do 
+  case "$filename" in
+    *wav)
+    $BASIC_PLAY_CMD $filename >/dev/null 2>&1;
+    ;;
+    *mp3)
+    $MP3_PLAY_CMD $filename >/dev/null 2>&1;
+    ;;
+    *)
+    echo "I don't know the file extension here: $filename"
+    ;;
+  esac
+done
 
 exit 0
 
-- 
2.34.1