ports/153755: [patch] multimedia/mencoder: and configurable fonconfig dependency

[Eygene Ryabinkin]

>Number:         153755
>Category:       ports
>Synopsis:       [patch] multimedia/mencoder: and configurable fonconfig dependency
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Fri Jan 07 10:20:00 UTC 2011
>Closed-Date:
>Last-Modified:
>Originator:     Eygene Ryabinkin
>Release:        FreeBSD 9.0-CURRENT amd64
>Organization:
Code Labs
>Environment:

Sysmtem: FreeBSD 9.0-CURRENT amd64

>Description:

As was discussed in [1], it will be good to add the fontconfig dependency
to the multimedia/mencoder to allow fontconfig'ish font specifications.

>How-To-Repeat:

Look at
[1] http://lists.freebsd.org/pipermail/freebsd-ports/2010-December/065147.html
[2] http://forums.freebsd.org/showthread.php?t=18287
Try to use '-font Verdana' at mencoder command line.

>Fix:

The following patch will introduce the fonconfig dependency
that is off by-default.
--- add-fontconfig-dependency.diff begins here ---
>From 1bb1e04ebbafbb58b05bec1c0d5c05e8e9183c78 Mon Sep 17 00:00:00 2001
From: Eygene Ryabinkin <rea-fbsd at codelabs.ru>
Date: Fri, 7 Jan 2011 00:43:06 +0300

Reference: http://lists.freebsd.org/pipermail/freebsd-ports/2010-December/065147.html
Signed-off-by: Eygene Ryabinkin <rea-fbsd at codelabs.ru>
---
 multimedia/mencoder/Makefile |   10 ++++++++--
 1 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/multimedia/mencoder/Makefile b/multimedia/mencoder/Makefile
index 74ce127..167c7e6 100644
--- a/multimedia/mencoder/Makefile
+++ b/multimedia/mencoder/Makefile
@@ -6,13 +6,14 @@
 
 PORTNAME=	mencoder
 PORTVERSION=	${MPLAYER_PORT_VERSION}
-PORTREVISION=	0
+PORTREVISION=	1
 COMMENT=	Convenient video file and movie encoder
 RESTRICTED=	Port has restricted dependencies
 
 .include "${.CURDIR}/../mplayer/Makefile.shared"
 
 OPTIONS=	DEBUG "Include debug symbols in mencoder's binary" off
+OPTIONS+=	FONTCONFIG "Enable fontconfig support" off
 OPTIONS+=	RTCPU "Allow runtime CPU detection" on
 OPTIONS+=	OCFLAGS "Use optimized compiler flags" on
 OPTIONS+=	IPV6 "Include inet6 network support" on
@@ -67,7 +68,6 @@ CONFIGURE_ARGS+=	--disable-mplayer \
 			--disable-libdca \
 			--disable-x11 \
 			--disable-vdpau \
-			--disable-fontconfig \
 			--disable-rtc \
 			--disable-arts \
 			--disable-esd \
@@ -86,6 +86,12 @@ ALL_TARGET=	mencoder
 
 .include "${.CURDIR}/../mplayer/Makefile.options"
 
+.if defined(WITH_FONTCONFIG)
+LIB_DEPENDS+=	fontconfig.1:${PORTSDIR}/x11-fonts/fontconfig
+.else
+CONFIGURE_ARGS+=	--disable-fontconfig
+.endif
+
 .if defined(WITH_TWOLAME)
 LIB_DEPENDS+=	twolame.0:${PORTSDIR}/audio/twolame
 .else
-- 
1.7.3.4
--- add-fontconfig-dependency.diff ends here ---

Tinderbox builds are fine, port with enabled fonconfig allow
for the specifications like 'Verdana:size=24' and alike.
>Release-Note:
>Audit-Trail:
>Unformatted: