Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 414827 - media-sound/mpd should depend on media-libs/alsa-lib
Summary: media-sound/mpd should depend on media-libs/alsa-lib
Status: VERIFIED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Christoph Mende (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-06 06:11 UTC by nzqr
Modified: 2012-05-31 14:20 UTC (History)
2 users (show)

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


Attachments
emerge --info output (emerge_info.txt,3.42 KB, text/plain)
2012-05-18 15:53 UTC, nzqr
Details

Note You need to log in before you can comment on or make changes to this bug.
Description nzqr 2012-05-06 06:11:11 UTC
media-sound/mpd-0.16.8 RDEPENDs on media-sound/alsa-utils, but grep-ing doesn't show any usage of alsa utils in it's sources. But it links with libasound:
$ readelf -d `which mpd` |grep libasound
 0x0000000000000001 (NEEDED)             Shared library: [libasound.so.2]

so it rather should RDEPEND on media-libs/alsa-lib.

Reproducible: Always
Comment 1 nzqr 2012-05-18 15:53:02 UTC
Created attachment 312203 [details]
emerge --info output
Comment 2 Christoph Mende (RETIRED) gentoo-dev 2012-05-27 07:45:32 UTC
+  27 May 2012; Christoph Mende <angelos@gentoo.org> mpd-0.16.8.ebuild:
+  Depend on media-libs/alsa-lib (bug #414827)
Comment 3 nzqr 2012-05-27 15:45:22 UTC
Ok, but depend on media-sound/alsa-utils is unnecessary.
Comment 4 Markos Chandras (RETIRED) gentoo-dev 2012-05-31 08:58:50 UTC
Please do not reopen and rename old bugs. Open a new one
Comment 5 nzqr 2012-05-31 10:22:32 UTC
Why? Someone already renamed it in wrong way; i posted as:
>media-sound/mpd-0.16.8: suspect RDEPENDs
they renamed to
>media-sound/mpd should depend on media-libs/alsa-lib
that messed up things.
Comment 6 Christoph Mende (RETIRED) gentoo-dev 2012-05-31 12:06:11 UTC
The init script makes use of alsa-utils.
Comment 7 nzqr 2012-05-31 12:41:36 UTC
>The init script makes use of alsa-utils
Where?


#!/sbin/runscript
# Copyright 1999-2010 Gentoo Foundation
# Distributed under the terms of the GNU General Public License v2
# $Header: /var/cvsroot/gentoo-x86/media-sound/mpd/files/mpd.init,v 1.1 2010/09/10 07:55:07 angelos Exp $

depend() {
	need localmount
	use net netmount nfsmount alsa esound pulseaudio
} 

checkconfig() {
	if ! [ -f /etc/mpd.conf ]; then
		eerror "Configuration file /etc/mpd.conf does not exist."
		return 1
	fi

	return 0
}

start() {
	checkconfig || return 1

	ebegin "Starting Music Player Daemon" 
	start-stop-daemon --start --quiet --background --exec /usr/bin/mpd --pidfile /var/run/mpd.pid --make-pidfile -- --no-daemon /etc/mpd.conf 2>/dev/null
	eend $?
}

stop() {
	ebegin "Stopping Music Player Daemon"
	start-stop-daemon --stop --exec /usr/bin/mpd --pidfile /var/run/mpd.pid
	eend $? 
}
Comment 8 Christoph Mende (RETIRED) gentoo-dev 2012-05-31 14:20:07 UTC
(In reply to comment #7)
> >The init script makes use of alsa-utils
> Where?
> 
> 
> #!/sbin/runscript
> # Copyright 1999-2010 Gentoo Foundation
> # Distributed under the terms of the GNU General Public License v2
> # $Header: /var/cvsroot/gentoo-x86/media-sound/mpd/files/mpd.init,v 1.1
> 2010/09/10 07:55:07 angelos Exp $
> 
> depend() {
> 	need localmount
> 	use net netmount nfsmount alsa esound pulseaudio
                                  ^^^^ right here

And please don't tell me "that's just a use". I will not remove the dependency. If you really can't live a package that's ~2 MB on disk, put it in package.provided.