retrosmc icon indicating copy to clipboard operation
retrosmc copied to clipboard

install-retrosmc.sh "add fix to config.txt for sound" bug

Open FraserChapman opened this issue 8 years ago • 1 comments
trafficstars

The condition at - https://github.com/mcobit/retrosmc/blob/master/install-retrosmc.sh#L62 is a bit too simple, in that many configurations have the line 'dtparam=audio=on' but have it commented out, i.e. '#dtparam=audio=on'

As it is the grep will match these commented out line and then insert 'dtparam=audio=on' - thus breaking settings for any sound cards, overlays, etc.

I would suggest simply adding the caret to the command so that only lines starting match. e.g.

grep -e "^dtparam=audio=on'" /boot/config.txt

FraserChapman avatar Feb 18 '17 23:02 FraserChapman

Thanks for the suggestion. And of course you are right. I will test and fix this tomorrow.

mcobit avatar Feb 27 '17 07:02 mcobit