No mpd connection
Bug #518189 reported by
Olivier Samyn
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| CoverGloobus |
Fix Released
|
Medium
|
Alexandr Grigorcea | ||
Bug Description
Just tried covergloobus today, and found that using the latest trunk code, it's impossible to connect cg to MPD.
This commit moved the "set_config" call after the player initialisation:
http://
Moving it back before, let's the MPD player use the actual configuration and connect to the MPD server.
So here is a simple patch that solves this.
Related branches
lp:~cahr-gr/covergloobus/mpd
- BadChoice: Pending requested
-
Diff: 35 lines (+5/-2)2 files modifiedsrc/covergloobus.py (+1/-1)
src/players/MPD.py (+4/-1)
Revision history for this message
| Olivier Samyn (olivier-oleastre) wrote | #1 |
Revision history for this message
| Alexandr Grigorcea (cahr-gr) wrote | #2 |
| Changed in covergloobus: | |
| assignee: | nobody → Alexandr Grigorcea (cahr-gr) |
| importance: | Undecided → Medium |
| status: | New → Fix Committed |
| Changed in covergloobus: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
thanks, Olivier
I moved it out because if MPD is not running when CG starts, config var will not be set, but placed it in a wrong place
fixed it , should be ok when merged into trunk