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

Add option to disable BT and add storage directory #50

Closed owenashurst closed 5 years ago

owenashurst commented 5 years ago

I'd like the option to disable bluetooth and add a storage path to where my music will live on the Pi instead of a USB stick.

I've tried running mpradio on my Pi 2 which doesn't have bluetooth and it fails;

pi@raspberrypi:~ $ mpradio freq: 88.30 btGain: 1.7 storageGain: 1 persistentPlaylist: 1 resumePlayback: 1 shuffle: 1 rdsUpdateInterval: 3 rdsCharsJump: 6 Output: fm Format: mp3 -o -iname *.flac -o -iname *.wav implementation: pi_fm_adv Treble: 0 no bluetooth device provided, playing mp3s... terminate called without an active exception CONTROL_PIPE_SETUPAborted

Any ideas?

morrolinux commented 5 years ago

just add your songs in a folder called /pirateradio man :)

owenashurst commented 5 years ago

just add your songs in a folder called /pirateradio man :)

Is that root? /pirateradio?

morrolinux commented 5 years ago

yup

owenashurst commented 5 years ago

Working now, thanks! However, the "rdsPattern" seems to be ignored in the config. If I set it to "TEST" it won't show on the receiver as "TEST". Both the titles on the receiver show the "Artist name - Song name" - I would have thought one of these should change?

owenashurst commented 5 years ago

I'll have a go at implementing this and submit a PR.

morrolinux commented 5 years ago

I don't get what you're talking about? could you be a little more specific?

owenashurst commented 5 years ago

The station name (PS) is also using the song name as well as the radio text (RT). I thought the station name could be configurable but it doesn't seem like it can. In the code, the station name is hard coded to "songName" for some reason.

Could this be made configurable please?

owenashurst commented 5 years ago

I've added a fix for this in a branch. Could you give me permissions to make a pull request please?

EDIT: Haven't used Github in a while, realised I had to fork. I've submitted a PR for this fix here: https://github.com/morrolinux/mpradio/pull/51