ports/123763: maintainer update: multimedia/subtitlecomposer

[Max Brazhnikov]

>Number:         123763
>Category:       ports
>Synopsis:       maintainer update: multimedia/subtitlecomposer
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          maintainer-update
>Submitter-Id:   current-users
>Arrival-Date:   Sat May 17 14:00:08 UTC 2008
>Closed-Date:
>Last-Modified:
>Originator:     Max Brazhnikov
>Release:        FreeBSD 7.0-STABLE i386
>Organization:
>Environment:
FreeBSD luna.dio.ru 7.0-STABLE FreeBSD 7.0-STABLE #0: Thu Apr 17 12:14:46 MSD 2008 makc at luna.dio.ru:/usr/obj/usr/src/sys/LUNA i386
>Description:
1) make MPlayer support optional
2) add GStreamer support
3) fix Xine support (required XCB support in libxine, see ports/123761)
>How-To-Repeat:

>Fix:


Patch attached with submission follows:

diff -ruN --exclude=CVS /usr/ports/multimedia/subtitlecomposer/Makefile /home/makc/porting/ports/multimedia/subtitlecomposer/Makefile
--- /usr/ports/multimedia/subtitlecomposer/Makefile	2008-04-24 20:24:55.000000000 +0400
+++ /home/makc/porting/ports/multimedia/subtitlecomposer/Makefile	2008-05-17 17:42:33.000000000 +0400
@@ -7,6 +7,7 @@
 
 PORTNAME=	subtitlecomposer
 PORTVERSION=	0.4.1
+PORTREVISION=	1
 CATEGORIES=	multimedia
 MASTER_SITES=	SF
 MASTER_SITE_SUBDIR=subcomposer
@@ -14,23 +15,43 @@
 MAINTAINER=	makc at issp.ac.ru
 COMMENT=	Subtitle editor for KDE
 
-RUN_DEPENDS=	mplayer:${PORTSDIR}/multimedia/mplayer
-
 USE_BZIP2=	yes
 USE_AUTOTOOLS=	libtool:15
 USE_KDELIBS_VER=3
 GNU_CONFIGURE=	yes
-CONFIGURE_ARGS=	--without-gstreamer
+WANT_GSTREAMER=	yes
 INSTALLS_ICONS=	yes
 
-OPTIONS=	XINE "Enable Xine backend" off
+OPTIONS=	MPLAYER "Enable MPlayer backend (recommended)" on \
+		XINE "Enable Xine backend" off \
+		GSTREAMER "Enable GStreamer backend" off
 
 .include <bsd.port.pre.mk>
 
+.if defined(WITH_MPLAYER)
+RUN_DEPENDS+=	mplayer:${PORTSDIR}/multimedia/mplayer
+.endif
+
 .if defined(WITH_XINE)
-LIB_DEPENDS+=	xine.1:${PORTSDIR}/multimedia/libxine
+
+. if exists(${LOCALBASE}/bin/xine-config)
+XINE_PLUGINDIR!=	xine-config --plugindir
+. endif
+
+. if defined(XINE_PLUGINDIR) && !exists(${XINE_PLUGINDIR}/xineplug_vo_out_xcbxv.so)
+IGNORE=		needs XCB support in libxine
+. endif
+
+LIB_DEPENDS+=	xine.1:${PORTSDIR}/multimedia/libxine \
+		xcb.1:${PORTSDIR}/x11/libxcb
 .else
-CONFIGURE_ARGS+=--without-xine
+CONFIGURE_ARGS+=--without-xine --without-xcb
+.endif
+
+.if defined(WITH_GSTREAMER)
+USE_GSTREAMER+=	yes
+.else
+CONFIGURE_ARGS=	--without-gstreamer
 .endif
 
 .include <bsd.port.post.mk>


>Release-Note:
>Audit-Trail:
>Unformatted: