Fix #132 - remove hardcoded play commands #148
Conversation
|
Results:
|
|
As per noted from sgee https://chat.mycroft.ai/community/pl/wrk6x1som3bu8kebcn9rphzx6r, there are a lot of further complexities here I don't know how to handle Namely: enclosure-picroft/home/pi/auto_run.sh Line 291 in afd739e Perhaps it is fine to remove from this file (hard coded crap), and let the |
|
Maybe we wait and review sgee's large refactor of these scripts first. I think there's a lot of improvement possible here including moving away from sed to a more purposeful tool like jq. |
|
It would be really super duper cool for this fix to land, I fell on my face twice setting up two Picrofts as the "bug" shows itself possibly very differently. The first time I almost lost my sanity because Mycroft just wouldn't say anything as the hardcoded device did not match mine. I messed with way to many config files in the entire linux audio stack until figuring that out. The second time around it seemed like Mycroft could not adjust it's volume. I saw it moving the pluseaudio master volume up and down, it just had no effect. In the end it turned out that Mycroft was bypassing pulseaudio by having the device hardcoded in that system config file. |
Removes hardcoded play commands from Picroft. These were added as a work around for issues many moons ago. It is believed that these no longer exist in Debian Buster.
Fixes #132
How to test this PR
It would be helpful if anyone running Picroft could remove these lines from their system config, reboot, and verify that output continues as expected. Please comment on this PR with your findings good or bad including the audio devices you are using.
signed a CLA (Contributor Licensing Agreement)