[Bugfix] Handle card index properly #2

README.md

@@ -78,6 +78,7 @@ An already started tidal-connect container should start working immediately, at
 
 Date|Comment
 :---|:---
+2023-07-07|Fixed asound.conf generation from card index, see issue ([#2](https://github.com/GioF71/tidal-connect/issues/2))
 2023-06-02|First unfolding seems to be working
 2023-06-02|Some effort to avoid resampling
 2023-06-02|MQA passthrough defaults to `false`

bin/entrypoint.sh

@@ -45,7 +45,10 @@ if [[ "${card_index}" == "-1" && -n "${card_name}" ]]; then
          fi
       fi
    done
-elif [[ -z "${card_index}" ]]; then
+elif [[ -n "${card_index}" ]]; then
+    echo "Set card_index=[$card_index]"
+    write_audio_config $card_index
+else
     echo "Set default card_index=[$DEFAULT_CARD_INDEX]"
     write_audio_config $DEFAULT_CARD_INDEX
 fi

configure.sh

@@ -1,5 +1,8 @@
 #!/bin/bash
 
+# error codes
+# 1 cannot specify both index and name
+
 set -ex
 
 ENV_FILE=.env
@@ -18,6 +21,11 @@ do
     esac
 done
 
+if [[ -n "${card_index}" && -n "${card_name}" ]]; then
+    echo "Cannot specify both index and name for audio card"
+    exit 1
+fi
+
 echo "card_index=[$card_index]"
 echo "card_name=[$card_name]"
 
@@ -53,4 +61,5 @@ if [[ -n "${card_name}" ]]; then
     echo "CARD_NAME=${card_name}" >> $ENV_FILE
 elif [[ -n "${card_index}" ]]; then
     echo "CARD_INDEX=${card_index}" >> $ENV_FILE
+    echo "CARD_NAME=NOT_SET" >> $ENV_FILE
 fi