eworm-de / mpd-notification

Notify about tracks played by mpd
GNU General Public License v3.0
86 stars 23 forks source link

TIMEOUT only works from command line #18

Closed chowbok closed 7 years ago

chowbok commented 7 years ago

I tried setting the TIMEOUT to various values in my config, but it doesn't change. Setting it via -t works, however. I know it's reading my config because it's connecting to the correct server, etc.

eworm-de commented 7 years ago

The timeout from configuration file is read on line 255. I do not see a reason why this should fail... Make sure not to give extra -t on the command line, that overwrites the value from configuration file.

eworm-de commented 7 years ago

Just verified, works for me. Make sure to give just numeric value (in seconds). No precision, no suffix.

chowbok commented 7 years ago

Well, I've tested it and tested it, I swear it only works from the command line. Here's my config: host = squirrel port = 6600 music-dir = /squirrel/audio/ timeout = 30 scale = 500 I guess I'll just add the flag in the systemd file for now.

eworm-de commented 7 years ago

Ah, found the issue. I tested with small values and did not notice this is not exactly what was expected. In config file values of ms (like timeout = 30000) would work...

eworm-de commented 7 years ago

Make sure to update (or remove) config.h for a clean default value.