morrolinux / mpradio

Morrolinux's Pirate radio (PiFmRDS / PiFmAdv implementation with Bluetooth and mp3 support) - Stream music to your car's FM radio or use it as a Bluetooth speaker via headphone jack
GNU General Public License v3.0
107 stars 17 forks source link

A couple more fixes for bluetooth-only users #31

Closed Hurricos closed 6 years ago

Hurricos commented 6 years ago

RE: commit a13de5b, the message should have been "Test for existence before catting now_playing". The now_playing file is only available when playing from storage; if there are no files at all, or only a bluetooth connection, this would cause error messages which would end up in /var/log/daemon.log and /var/log/syslog, amounting to >500MB of messages in a 24-hour period :(

RE: commit de47dae, just making a backup of the local pirateradio.config file on reinstallation, to give users a chance to get it out of the way. Should probably either ask the user which configuration file to use, or not overwrite at all, but this is likely the least intrusive fix.

Thank you. Apologies for the piecemeal pull requests, but I figure they're useful changes.