Bug 177721

Summary: multimedia/mplayer: Missing dependency
Product: Ports & Packages Reporter: Walter Hurry <walterhurry>
Component: Individual Port(s)Assignee: Thierry Thomas <thierry>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
m.diff.txt none

Description Walter Hurry 2013-04-08 21:10:00 UTC 
multimedia/mplayer needs libXss.so.1 at runtime, so it should have a dependency on x11/libXScrnSaver, but it does not.

x11/libXScrnSaver should be added as a dependency.

Fix: 

Add x11/libXScrnSaver as a dependency of multimedia/mplayer
How-To-Repeat: Remove x11/libXScrnSaver. mplayer now errors out with "shared library not found"
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2013-04-08 21:10:16 UTC 
State Changed
From-To: open->feedback

Awaiting maintainers feedback (via the GNATS Auto Assign Tool)
Comment 2 Edwin Groothuis freebsd_committer freebsd_triage 2013-04-08 21:10:16 UTC 
Maintainer of multimedia/mplayer,

Please note that PR ports/177721 has just been submitted.

If it contains a patch for an upgrade, an enhancement or a bug fix
you agree on, reply to this email stating that you approve the patch
and a committer will take care of it.

The full text of the PR can be found at:
    http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/177721

-- 
Edwin Groothuis via the GNATS Auto Assign Tool
edwin@FreeBSD.org
Comment 3 Thomas Zander 2013-04-09 21:16:21 UTC 
I am not near my dev/test system right now, but attached patch should
do it. Can you verify?

Regards
Riggs
Comment 4 Walter Hurry 2013-04-10 15:58:26 UTC 
Works for me:

$ grep USE_XORG= /usr/ports/multimedia/mplayer/Makefile
USE_XORG=	x11 xscrnsaver
$ sudo pkg delete -y libXScrnSaver
The following packages will be deinstalled:

	libXScrnSaver-1.2.1

The deinstallation will free 34 kB
Deleting libXScrnSaver-1.2.1...
libXScrnSaver-1.2.1 is required by: mplayer-1.1.r20130308
$

Walter
Comment 5 Pawel Pekala freebsd_committer freebsd_triage 2013-05-02 14:05:01 UTC 
This patch will not work for X11=3Doff, binary will still link to
libXScrnSaver. Configure script have switch to control this:
 --(enable|disable)-xss which defaults to autodetection.

--=20
pozdrawiam / with regards
Pawe=B3 P=EAkala
Comment 6 Thierry Thomas freebsd_committer freebsd_triage 2014-06-01 07:55:55 UTC 
State Changed
From-To: feedback->analyzed


Feedback received. See PR ports/185142.
Comment 7 Thierry Thomas freebsd_committer freebsd_triage 2014-06-01 07:55:55 UTC 
Responsible Changed
From-To: freebsd-ports-bugs->thierry


Take it.
Comment 8 Thierry Thomas freebsd_committer freebsd_triage 2014-06-01 07:55:55 UTC 
State Changed
From-To: analyzed->closed


Dependency registration on screensaver has been fixed when upgradingt to 
1.1.20131223 (see PR ports/185142). 

Thanks for the report!