Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 670130 - media-sound/upmpdcli should not RDEPEND on media-sound/mpd
Summary: media-sound/upmpdcli should not RDEPEND on media-sound/mpd
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Erik Mackdanz
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-11-02 07:02 UTC by BT
Modified: 2019-02-10 00:39 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description BT 2018-11-02 07:02:09 UTC
Upmpdcli does not require MPD to be running on the same host as it can be configured to connect to a different host running MPD. I have confirmed this by removing `need mpd` from the upmpdcli init script.

I'm currently running Gebera and upmpdcli on a Gentoo machine and MPD on a different Linux host that doesn't have upmpdcli available. The current upmpdcli ebuild forces MPD to be installed and running on my Gentoo machine even though it's not required.
Comment 1 BT 2018-11-03 22:43:31 UTC
Just to elaborate on how I ran upmpdcli without mpd running on the same host.

* Stop upmpdcli and mpd
* Remove 'need mpd' from /etc/init.d/upmpdcli
* Set the 'mpdhost' option to a different host in /etc/upmpdcli.conf
* Start upmpdcli
* Ran `pgrep mpd` to ensure mpd wasn't running

If you decide to drop the RDEPEND on mpd, it would be useful to mention in the post install message that if ruining mpd on the same host as upmpdcli, then mpd needs to have curl support enabled.
Comment 2 Larry the Git Cow gentoo-dev 2019-02-09 15:53:01 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=40f04f1ef95b0aa1b5e2e30e3bd9649441e9f5da

commit 40f04f1ef95b0aa1b5e2e30e3bd9649441e9f5da
Author:     Erik Mackdanz <stasibear@gentoo.org>
AuthorDate: 2019-02-09 15:52:39 +0000
Commit:     Erik Mackdanz <stasibear@gentoo.org>
CommitDate: 2019-02-09 15:52:39 +0000

    media-sound/upmpdcli: Bump to 1.4.0
    
    Closes: https://bugs.gentoo.org/670130
    Closes: https://bugs.gentoo.org/673552
    Signed-off-by: Erik Mackdanz <stasibear@gentoo.org>
    Package-Manager: Portage-2.3.59, Repoman-2.3.12

 media-sound/upmpdcli/Manifest              |  1 +
 media-sound/upmpdcli/files/upmpdcli.initd  | 14 ++++++++
 media-sound/upmpdcli/upmpdcli-1.4.0.ebuild | 51 ++++++++++++++++++++++++++++++
 3 files changed, 66 insertions(+)
Comment 3 Erik Mackdanz gentoo-dev 2019-02-09 15:54:17 UTC
This is fixed in 1.4.0.  Thanks for calling this out.
Comment 4 BT 2019-02-10 00:39:03 UTC
Thank you for fixing this bug.

There is a minor error in the following einfo line:

einfo "mpdinstance on the same host (https://bugs.gentoo.org/670130)."

'mpdinstance' should be 'mpd instance'.