Patches look fine, any reason not to merge the first one? I guess it's debatable whether rhythmbox-plugins should be a dependency or a recommendation, but most definitely, the context panel is missing from rhythmbox-plugins.
On a related note, recent changes to trusty's rhythmbox include:
47. By Sebastien Bacher on 2014-03-20
* debian/control.in:
- restore the recommends on rhythmbox-plugins, that was dropped by
error when merging on Debian. Without those integration to the sound
indicator is not working (that needs the mpris plugin to be loaded)
(lp: #1289024)
Patches look fine, any reason not to merge the first one? I guess it's debatable whether rhythmbox-plugins should be a dependency or a recommendation, but most definitely, the context panel is missing from rhythmbox-plugins.
On a related note, recent changes to trusty's rhythmbox include:
47. By Sebastien Bacher on 2014-03-20
* debian/control.in:
- restore the recommends on rhythmbox-plugins, that was dropped by
error when merging on Debian. Without those integration to the sound
indicator is not working (that needs the mpris plugin to be loaded)
(lp: #1289024)